fix(mergify): use actual check context names for Dependabot rules#571
Conversation
|
Warning Review limit reached
More reviews will be available in 30 minutes and 50 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the Mergify configuration in .github/mergify.yml to simplify the names of the required success checks (e.g., changing CI / check to check). Feedback suggests that the second rule for auto-merging security updates is entirely redundant and can be removed to simplify the configuration.
| - check-success=check | ||
| - check-success=lint | ||
| - check-success=validate |
There was a problem hiding this comment.
The rule 'Auto-merge Dependabot security updates on develop' (lines 16-29) is entirely redundant. Any pull request that satisfies its conditions (including having both 'area:dependencies' and 'meta:dependabot-security' labels) will already satisfy the conditions of the first rule 'Auto-merge Dependabot dependency updates on develop' (lines 2-14) and be merged using the same squash method. Consider removing the redundant second rule to simplify the configuration.
🔍 Reviewer Summary for PR #571CI Status: ✅ Recommendations
|
Summary\n- update Mergify check-success conditions to match actual check run names used on Dependabot PRs\n- replace prefixed context strings with check, lint, and validate\n\n## Why\nDependabot PRs #532 and #533 passed checks but remained open because Mergify conditions referenced non-matching context names.\n\n## Validation\n- npm run lint:yaml\n- npm run test:js (pre-commit)\n