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

ci: added priority rules for high priority #25

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

frazarshad
Copy link
Collaborator

replaces special high priority queues with high priority rules

@frazarshad frazarshad self-assigned this Oct 14, 2024
mergify bot added a commit to Agoric/agoric-sdk that referenced this pull request Oct 17, 2024
## Description

Mergify has recently added priority rules feature to mergify configuration

This allows us to define priority rules of each item in our merge queue, independent of the the priority of the queue

What this means for us is that we can remove all the separate high priority queues and simplify our configuration

Also created a [PR in Agoric/mergify-experiments](Agoric/mergify-experiements#25) that the updates made here

### Testing

This configuration has been tested out on [frazarshad/mergify-experiments ](https://github.com/frazarshad/mergify-experiements)
![image](https://github.com/user-attachments/assets/3a88ec98-62ce-4373-9954-36dc99746017)

Similarly I also tested that the queues don't have any priority base on their order in the configuration and don't preempt each other
![image](https://github.com/user-attachments/assets/5332df09-973e-497b-b44f-8e918444b2f4)
@frazarshad frazarshad added the automerge:rebase Automatically rebase updates, then merge label Oct 17, 2024
@frazarshad frazarshad force-pushed the fraz/mergify-priority-rules branch from f16e7da to 9b92867 Compare October 17, 2024 21:48
@mergify mergify bot merged commit f7ca954 into main Oct 17, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants