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.

Sign up for GitHub

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

Jump to bottom

refactor(mergify): streamline queue and priority rules #9068

Merged
merged 1 commit into from
Dec 4, 2024
Merged

refactor(mergify): streamline queue and priority rules #9068

merged 1 commit into from
Dec 4, 2024

Conversation

gustavovalverde
Copy link
Member

Motivation

Our Mergify configuration file missed some adjustments to adapt to their new configuration

Specifications & References

Solution

  • Removed deprecated actions and simplified default queue rules.
  • Adjusted batch sizes for urgent and batched queues.
  • Consolidated priority rules to enhance clarity and efficiency.
  • Updated conditions for moving pull requests to different queues based on labels and reviews.

Tests

  • We can only fully test after merging

PR Author's Checklist

  • The PR name will make sense to users.
  • The PR provides a CHANGELOG summary.
  • The solution is tested.
  • The documentation is up to date.
  • The PR has a priority label.

PR Reviewer's Checklist

  • The PR Author's checklist is complete.
  • The PR resolves the issue.

@gustavovalverde
refactor(mergify): streamline queue and priority rules
9ebe7ed 
- Removed deprecated actions and simplified default queue rules.
- Adjusted batch sizes for urgent and batched queues.
- Consolidated priority rules to enhance clarity and efficiency.
- Updated conditions for moving pull requests to different queues based on labels and reviews.
@gustavovalverde gustavovalverde added C-bug Category: This is a bug A-devops Area: Pipelines, CI/CD and Dockerfiles I-integration-fail Continuous integration fails, including build and test failures P-Critical 🚑 labels Dec 4, 2024
@gustavovalverde gustavovalverde self-assigned this Dec 4, 2024
@gustavovalverde gustavovalverde requested a review from a team as a code owner December 4, 2024 14:34
@gustavovalverde gustavovalverde requested review from oxarbitrage and removed request for a team December 4, 2024 14:34
@github-actions github-actions bot added the C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG label Dec 4, 2024
@gustavovalverde gustavovalverde merged commit e72c0fe into main Dec 4, 2024
97 checks passed
@gustavovalverde gustavovalverde deleted the update-mergify-conf branch December 4, 2024 14:52
@arya2 arya2 mentioned this pull request Dec 5, 2024
Merged
45 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mpguerra mpguerra mpguerra approved these changes

@oxarbitrage oxarbitrage Awaiting requested review from oxarbitrage oxarbitrage is a code owner automatically assigned from ZcashFoundation/devops-reviewers

Assignees

@gustavovalverde gustavovalverde

Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles C-bug Category: This is a bug C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG I-integration-fail Continuous integration fails, including build and test failures P-Critical 🚑
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gustavovalverde @mpguerra