Auto-merge only Dependabot's own minor/patch group PRs - #69
Conversation
The guard let through any PR whose update-type was not semver-major. For a requirement range with no lockfile, Dependabot emits no update-type, and fetch-metadata reported Claude-Patent-Creator #82 (mcp widened from <2.0.0 to <3.0.0) as version-update:semver-patch. A major landed on main by itself and broke that server at import. Dependabot decides group membership from versions it can actually compare, so the group name is the trustworthy signal. The guard now merges only PRs in a *-minor-patch group. Majors, range widens, and anything Dependabot could not classify wait for a person. - security-minor-patch group so minor/patch security fixes still flow on their own. - actions-minor-patch group so GitHub Actions bumps keep auto-merging.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughDependabot now groups minor and patch updates for pip and GitHub Actions. The auto-merge workflow merges only updates in groups whose names end with ChangesDependabot automation
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to Dependabot now auto-merges only grouped minor and patch updates, preventing major, security, and unclassified updates from merging automatically. No current merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Why
In Claude-Patent-Creator, Dependabot PR #82 widened a pin from
<2.0.0to<3.0.0and was auto-merged, breaking the server at import. The auto-merge guard here is the same one: it checksupdate-type != semver-major, but for a requirement range with no lockfile Dependabot writes no update-type, and fetch-metadata reported that 1.x to 2.x widen asversion-update:semver-patch. A version-type guard cannot be trusted for those PRs.Change
Dependabot decides group membership from versions it can actually compare, so the group name is the honest signal. The workflow now auto-merges only PRs whose
dependency-groupends inminor-patch. Majors, range widens, and anything Dependabot could not classify sit open until a person merges them.security-minor-patch(applies to security updates) so minor/patch security fixes still merge on their own.actions-minor-patchso GitHub Actions bumps keep auto-merging.Summary by CodeRabbit