Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix operation injector script #4114

Merged
merged 1 commit into from
Jun 21, 2023
Merged

Fix operation injector script #4114

merged 1 commit into from
Jun 21, 2023

Conversation

litchipi
Copy link
Contributor

Before this PR, the command cargo check --features "bootstrap_server deadlock_detection op_spammer" failed, now it passes

(Note: Not tested yet in real case, but will do before merge)

@litchipi litchipi requested a review from AurelienFT June 19, 2023 14:37
Signed-off-by: Litchi Pi <litchi.pi@proton.me>
@litchipi litchipi force-pushed the fix_operation_spammer branch from 31ab073 to 9fe10ab Compare June 20, 2023 15:01
@litchipi
Copy link
Contributor Author

@AurelienFT Should I force bors to force-validate the CI for every single PR now ? :-(

@AurelienFT
Copy link
Contributor

@AurelienFT Should I force bors to force-validate the CI for every single PR now ? :-(

I don't understand your question. Bors is just a bot that help to merge. You can type "bors merge" in a GitHub comment and he will merge this pr

@litchipi
Copy link
Contributor Author

bors merge

@bors
Copy link
Contributor

bors bot commented Jun 21, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit cbad2db into main Jun 21, 2023
@AurelienFT AurelienFT deleted the fix_operation_spammer branch October 23, 2023 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants