Skip to content

chore: mergify does no longer support queue_conditions != merge_condi…#5891

Merged
cplee merged 1 commit intomasterfrom
ChristopherHX-patch-1
Aug 8, 2025
Merged

chore: mergify does no longer support queue_conditions != merge_condi…#5891
cplee merged 1 commit intomasterfrom
ChristopherHX-patch-1

Conversation

@ChristopherHX
Copy link
Contributor

…tions for require status checks to be up to date

Configuration not compatible with a branch protection setting
The branch protection setting Require branches to be up to date before merging is not compatible with max_parallel_checks>1, queue_conditions != merge_conditions and must be unset.

…tions for require status checks to be up to date

> Configuration not compatible with a branch protection setting
The branch protection setting Require branches to be up to date before merging is not compatible with max_parallel_checks>1, queue_conditions != merge_conditions and must be unset.
@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2025

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 1 0 0.0s
✅ REPOSITORY gitleaks yes no 2.76s
✅ REPOSITORY git_diff yes no 0.01s
✅ REPOSITORY grype yes no 12.82s
✅ REPOSITORY secretlint yes no 1.31s
✅ REPOSITORY trivy-sbom yes no 0.39s
✅ REPOSITORY trufflehog yes no 3.79s
⚠️ YAML prettier 1 1 0.26s
✅ YAML v8r 1 0 1.54s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@ChristopherHX
Copy link
Contributor Author

ChristopherHX commented Aug 1, 2025

Actually idk if this resolves the problem

https://github.com/nektos/act/runs/47170098045

The mergify config editor can successfully parse this change and shows that merge and queue conditions are the same

ref #5889

@codecov
Copy link

codecov bot commented Aug 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.78%. Comparing base (bd4bc99) to head (3d4017b).
⚠️ Report is 20 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5891      +/-   ##
==========================================
+ Coverage   74.65%   74.78%   +0.13%     
==========================================
  Files          73       73              
  Lines       11139    11190      +51     
==========================================
+ Hits         8316     8369      +53     
+ Misses       2186     2185       -1     
+ Partials      637      636       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ChristopherHX
Copy link
Contributor Author

Hi @cplee,

Manual Merge required, I can understand from a security point of view why mergify is not using the changed config to merge this PR.

@cplee cplee merged commit a78b3f3 into master Aug 8, 2025
11 of 12 checks passed
@cplee cplee deleted the ChristopherHX-patch-1 branch August 8, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants