ci(Mergify): configuration update#76
Conversation
Signed-off-by: UncleSp1d3r <unclesp1d3r@evilbitlabs.io>
|
Caution Review failedThe pull request is closed. Summary by CodeRabbit
WalkthroughA merge protection rule is added to Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 📃 Configuration Change RequirementsWonderful, this rule succeeded.Mergify configuration change
🟢 CI must passWonderful, this rule succeeded.All CI checks must pass or be legitimately skipped (path filtering). Matrix job names differ between running and skipped states, so the test-cross-platform rule uses an and/or pattern to handle both.
🟢 Do not merge outdated PRsWonderful, this rule succeeded.Make sure PRs are within 10 commits of the base branch before merging
|
There was a problem hiding this comment.
Pull request overview
Adds a Mergify merge_protections configuration to prevent merging PRs that are too far behind the main branch.
Changes:
- Introduces a “Do not merge outdated PRs” merge protection for PRs targeting
main. - Enforces a
#commits-behindthreshold viasuccess_conditions.
| - name: Do not merge outdated PRs | ||
| description: Make sure PRs are almost up to date before merging | ||
| if: | ||
| - base = main | ||
| success_conditions: | ||
| - "#commits-behind <= 10" |
There was a problem hiding this comment.
The protection description says PRs should be “almost up to date”, but the condition allows up to 10 commits behind. Either tighten the threshold (e.g., 0–1 commits behind) or adjust the wording so it matches the actual merge protection behavior.
🧪 CI InsightsHere's what we observed from your CI run for 92e4d5c. 🟢 All jobs passed!But CI Insights is watching 👀 |
This change has been made by @unclesp1d3r from the Mergify merge protections editor.