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

merge-queue: embarking main (744aca9) and #3582 together #3690

Closed
wants to merge 29 commits into from

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Mar 2, 2022

🙁 This combination of pull requests has failed checks. #3582 will be removed from the queue. 🙁

Branch main (744aca9) and #3582 are embarked together for merge.

This pull request has been created by Mergify to speculatively check the mergeability of #3582.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.

Required conditions of queue low for merge:

  • any of:
    • check-success=Clippy
    • check-neutral=Clippy
    • check-skipped=Clippy
  • any of:
    • check-success=Rustfmt
    • check-neutral=Rustfmt
    • check-skipped=Rustfmt
  • any of:
    • check-success=Test with fake activation heights
    • check-neutral=Test with fake activation heights
    • check-skipped=Test with fake activation heights
  • any of:
    • check-success=Test full validation sync from cached state
    • check-neutral=Test full validation sync from cached state
    • check-skipped=Test full validation sync from cached state
  • any of:
    • check-success=Test stable on macOS-latest
    • check-neutral=Test stable on macOS-latest
    • check-skipped=Test stable on macOS-latest
  • any of:
    • check-success=Test stable on ubuntu-latest
    • check-neutral=Test stable on ubuntu-latest
    • check-skipped=Test stable on ubuntu-latest
  • any of:
    • check-success=Test stable on windows-latest
    • check-neutral=Test stable on windows-latest
    • check-skipped=Test stable on windows-latest
  • any of:
    • check-success=Test all
    • check-neutral=Test all
    • check-skipped=Test all
  • #approved-reviews-by>=1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by=0 [🛡 GitHub branch protection]
  • #review-threads-unresolved=0 [🛡 GitHub branch protection]
  • check-success=Test all
  • check-success=Test with fake activation heights
  • check-success=Test full validation sync from cached state
  • check-success=Test stable zebra-state with fake activation heights on ubuntu-latest
  • check-success=Test stable on ubuntu-latest
  • check-success=Test stable on macOS-latest
  • check-success=Test stable on windows-latest
  • check-success=Coverage nightly
  • check-success=Clippy
  • check-success=Rustfmt

More informations about Mergify merge queue can be found in the documentation.

Mergify commands

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the queue rules

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

gustavovalverde and others added 29 commits February 18, 2022 06:10
error: Found argument '--nocapture' which wasn't expected, or isn't valid in this context
The mempool is only activated once, so we must check for that log first.
After mempool activation, the stop regex is logged at least once.
(It might be logged before as well, but we can't rely on that.)

When checking that the mempool didn't activate,
wait for the `zebrad` command to exit,
then check the entire log.
@mergify
Copy link
Contributor Author

mergify bot commented Mar 2, 2022

The pull request #3582 cannot be merged and has been disembarked

@mergify mergify bot closed this Mar 2, 2022
@mergify mergify bot deleted the mergify/merge-queue/main/3582 branch March 2, 2022 03:48
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