Skip to content

Conversation

@ozelalisen
Copy link
Member

@ozelalisen ozelalisen commented Aug 8, 2025

This change has been made by @ozelalisen from the Mergify Queue Rule Configurator.

Mergify does not sync branch protection rules. Overall disabling won't hurt because we have the same checks exist in mergify config already and branch protection rules won't even allow direct merge. It is already duplicate rule checking to include branch protection rules in mergify.

Signed-off-by: Alisen Berat Ozel <null>
@aws-cdk-automation aws-cdk-automation requested a review from a team August 8, 2025 13:11
@github-actions github-actions bot added the p2 label Aug 8, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Aug 8, 2025
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

@ozelalisen ozelalisen changed the title ci(Mergify): configuration update chore(ci): disable branch protection rules in mergify Aug 8, 2025
@ozelalisen ozelalisen added the pr/do-not-merge This PR should not be merged at this time. label Aug 8, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review August 8, 2025 13:19

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

mrgrain
mrgrain previously requested changes Aug 8, 2025
- -closed
- "#approved-reviews-by>=1"
- -approved-reviews-by~=author
# This is important! It makes the PR Linter work.
Copy link
Contributor

@mrgrain mrgrain Aug 8, 2025

Choose a reason for hiding this comment

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

Arguably these comments are very useful. Can we keep them pls?

Copy link
Member Author

Choose a reason for hiding this comment

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

That is why I kept it do-not-merge, will keep them

@ozelalisen ozelalisen removed the pr/do-not-merge This PR should not be merged at this time. label Aug 8, 2025
@ozelalisen ozelalisen requested a review from mrgrain August 8, 2025 15:12
@ozelalisen ozelalisen marked this pull request as draft August 8, 2025 15:46
@aws-cdk-automation aws-cdk-automation marked this pull request as ready for review August 8, 2025 16:50
@mergify
Copy link
Contributor

mergify bot commented Aug 11, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@alvazjor alvazjor merged commit 33703c7 into main Aug 11, 2025
18 checks passed
@alvazjor alvazjor deleted the mergify/ozelalisen/config-update branch August 11, 2025 14:32
@github-actions
Copy link
Contributor

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.

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants