chore(deps): consolidate dependabot into one grouped PR, minor/patch only - #1646
Conversation
…only Routine version updates are limited to minor and patch; semver-major bumps are ignored. The version-update: prefix scopes that ignore to version updates only, so a security advisory that can only be fixed by a major bump still opens a PR. Updates are grouped so they arrive as a single PR rather than one per dependency. Security advisories are grouped separately, per ecosystem. Mirrors the configuration proven on alaska-rcv, where it took 15 open Dependabot PRs down to 1. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FpuRSSAYEkdSSgnaFaoBKK
✅ Deploy Preview for bettervoting ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe Dependabot configuration now uses one weekly Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The configuration preserves Dependabot security updates, including fixes requiring major versions. No merge-blocking issue was identified. 🚥 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. A rabbit sees updates hop in one neat line Comment |
Part of an org-wide Dependabot consolidation. This repo currently has 30 open Dependabot PRs.
What changes
1. Routine updates are minor/patch only —
ignoredropsversion-update:semver-major. That prefix scopes the ignore to version updates, so a security advisory whose only fix is a major bump still opens a PR. Removing theupdate-types:key would make it a blanket ignore that silences security updates too; there's a comment in the file saying so.2. One PR instead of one per dependency.
multi-ecosystem-groupsmerges npm + bundler + docker + docker-compose + github-actions into a single weekly PR.3. Security advisories grouped separately via
*-securitygroups withapplies-to: security-updates— one PR per ecosystem rather than one per advisory.Already proven in this org
Running on alaska-rcv (15 PRs → 1), star-view (7 → 1), star-core (5 → 1), starpy and discord-bot.
On star-view, Dependabot took the majors down to the highest in-range version rather than skipping them:
react16.12.0→19.2.8 became 16.12.0→16.14.0,typescript3.8.3→7.0.2 became 3.8.3→3.9.10.Two rules learned the hard way, both encoded in the config:
open-pull-requests-limitmust not appear on an update entry belonging to a multi-ecosystem group — Dependabot rejects the whole file, and the published JSON schema does not catch it.applies-tokey, so it covers version updates only. Hence the separate*-securitygroups.What happens to the existing PRs
Mostly nothing — Dependabot cleans up after itself. Once this merges, it supersedes its own stale PRs and folds them into the grouped PR automatically. On star-view it closed 7 and replaced them with 1, unprompted.
The exception is PRs whose package has no in-range update left (an orphaned
actions/checkout 4→7, say). Those linger and need closing by hand.This repo has 14 open security alerts. Nothing here will be closed without first cross-referencing every PR against those alerts — see below.
🤖 Generated with Claude Code
https://claude.ai/code/session_01FpuRSSAYEkdSSgnaFaoBKK