Conversation
|
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 (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe Dependabot configuration now groups pip and GitHub Actions updates into one weekly pull request, ignores semver-major updates, limits each ecosystem to three open pull requests, and applies a ChangesDependabot update policy
Priority: ⚪ Not assessed Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to The Dependabot configuration supports the intended grouped weekly update policy and has no actionable merge risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
ec53956 to
a24be9c
Compare
|
Pushed a correction before this gets reviewed. The first version of this config dropped the per-ecosystem Fixed by keeping both, which the schema allows on the same entry:
Also added All 11 configs in the planned rollout now validate clean against the official schema. The validator was negative-tested first (bogus ecosystem name, unknown keys, wrong types, missing required keys — all six caught), so "valid" here means something. Runtime behaviour still needs this merged to confirm. |
Pilot for an org-wide Dependabot change. There are currently 106 open Dependabot PRs across 10 Equal-Vote repos; ~60% of them are major-version bumps that were deliberately left ungrouped by the previous config.
What changes
1. Minor and patch only for routine updates.
The
version-update:prefix scopes this to version updates. Security updates are filtered separately and are unaffected, so an advisory whose only fix is a major bump still opens a PR. Dropping theupdate-types:key would turn this into 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 pip and github-actions into a single weekly PR. (Single-ecosystem repos in the rollout get a plainpatterns: ["*"]group instead — cross-ecosystem grouping buys nothing there.)3.
open-pull-requests-limit10 → 3. Worth knowing this is a backstop, not the real control: it's counted per ecosystem entry rather than per repo, and security PRs are exempt from it entirely. The grouping in #2 is what actually holds the PR count down.Cooldown windows are carried over from the existing config unchanged.
Why this repo first
multi-ecosystem-groupsis GA, but its reference docs don't spell out how it interacts withignoreandopen-pull-requests-limit. A config Dependabot rejects means this repo silently stops getting updates, so this lands on starpy (2 ecosystems, 4 open PRs) first. Once it's merged and confirmed working, the same generator rolls out to the other 10 repos.This needs to merge before it can be verified — Dependabot only reads config from the default branch.
🤖 Generated with Claude Code
https://claude.ai/code/session_01XKNMLCKoVr8Xz1HePmDBXs
Summary by CodeRabbit