Skip to content

chore(mergify): configuration update #35203

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
Closed
wants to merge 1 commit into from
Closed

chore(mergify): configuration update #35203

wants to merge 1 commit into from

Conversation

@alvazjor
Copy link
Contributor

@alvazjor alvazjor commented Aug 11, 2025 •
edited
Loading

Fix Mergify queue blocking on removed CodeBuild checks

Problem

PRs are getting stuck in Mergify queues waiting for a removed CodeBuild check (AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv) that no longer exists in the CI pipeline. This causes indefinite blocking even when all actual required checks pass.

Solution

  • Added branch_protection_injection_mode: none to prevent Mergify from automatically injecting GitHub branch protection requirements that may reference historical/cached checks
  • Mergify will now only evaluate explicitly defined conditions in the configuration
  • Cleaned up formatting and removed outdated comments

Impact

  • Unblocks stuck PRs that meet all actual requirements
  • Prevents future issues when CI checks are removed or renamed
  • Maintains all existing merge safety requirements

@alvazjor
ci(Mergify): configuration update
2bb1291 
Signed-off-by: Jorge Diaz <null>
@aws-cdk-automation aws-cdk-automation requested a review from a team August 11, 2025 12:27
@github-actions github-actions bot added the p2 label Aug 11, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Aug 11, 2025
aws-cdk-automation
aws-cdk-automation previously requested changes Aug 11, 2025
View reviewed changes
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@alvazjor alvazjor changed the title ci(Mergify): configuration update chore(Mergify): configuration update Aug 11, 2025
@alvazjor alvazjor changed the title chore(Mergify): configuration update chore(mergify): configuration update Aug 11, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review August 11, 2025 12:32

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@alvazjor alvazjor closed this Aug 11, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 11, 2025

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@aws-cdk-automation aws-cdk-automation aws-cdk-automation left review comments

Assignees

No one assigned

Labels

contribution/core This is a PR that came from AWS. p2

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alvazjor @aws-cdk-automation