Repository navigation
Conversation
Extend the shared Python preset github>rishitank/.github:python, which groups and SHA-pins GitHub Actions bumps, handles vulnerability alerts immediately and keeps majors human-reviewed. Skip renovate[bot] and dependabot[bot] PRs in the Auto-Merge workflow so it does not override Renovate's per-update automerge policy. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0171XyyqTUH64gAi1AcerwjN
Renovate takes over version updates. Dependabot alerts stay enabled; they are a repository setting, independent of this file. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0171XyyqTUH64gAi1AcerwjN
|
@coderabbitai review |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: rishitank/animawatch/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. Summary by CodeRabbit
WalkthroughThe pull request removes Dependabot configuration, changes the Renovate preset and restricts the auto-merge job to eligible bot-authored pull requests. ChangesDependency update automation
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to No confirmed merge-blocking behavior is established. The effective Renovate Actions coverage and GitHub approval settings were not verified, so those remain operational uncertainties. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 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. The old update schedule fades away. Comment |
✅ Action performedReview finished.
|
main gained renovate.json from the Renovate onboarding PR (#34), which conflicts with this branch's renovate.json. The Update branch API cannot merge a conflicting branch, so this one-off workflow merges origin/main, keeps this branch's renovate.json (it extends the shared preset, which itself extends config:recommended), verifies the result and pushes the merge commit. It is deleted in a follow-up commit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0171XyyqTUH64gAi1AcerwjN
renovate.json was added on main by the Renovate onboarding PR (#34, extends config:recommended). This branch's version is kept: it extends the shared preset github>rishitank/.github:python, which itself extends config:recommended, so nothing from the onboarding file is lost.
It merged main into this branch, keeping this branch's renovate.json, and is no longer needed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0171XyyqTUH64gAi1AcerwjN
The fence skipped Renovate and Dependabot PRs, which left human and agent PRs as the only ones the workflow still armed. With the review-gate ruleset in place, a CodeRabbit approval satisfies the one required review, so an armed human/agent PR would merge as soon as CodeRabbit approved it, with no owner in the loop. PRs opened by a user account are now skipped as well. What remains armed is bot PRs other than Renovate and Dependabot (e.g. the release PR). Those still need an approval under review-gate, and CodeRabbit skips bot PRs, so they merge only after the owner approves. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0171XyyqTUH64gAi1AcerwjN
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
This PR is now ready for review; it had been left as a draft by mistake. Until it merges,
@rishitank: this is ready for you to merge. Also turn off Dependabot security updates in Settings → Code security, and keep Dependabot alerts on. |
Update, 25 Sep evening: main moved, and the auto-merge fence is tightened
Three things changed on
mainafter this PR was opened:renovate.jsonextendingconfig:recommended.auto-merge.ymlthen auto-merged chore: Configure Renovate #34, chore(deps): update dependency pytest to v9.0.3 [security] #35 (pytest) and chore(deps): update dependency pillow to v12.3.0 [security] #38 (Pillow) with no review.review-gateruleset now requires 1 approval. A CodeRabbit approval counts.What I did on this branch:
203fcca).renovate.jsonconflicted (add/add).github>rishitank/.github:python, which chains toconfig:recommended, so the onboarding file adds nothing.50fdf18).uv.lockchanges from chore(deps): update dependency pytest to v9.0.3 [security] #35 and chore(deps): update dependency pillow to v12.3.0 [security] #38.b6f56bd). It now also skips PRs opened by a user account (github.event.pull_request.user.type == 'Bot'is required).review-gate, CodeRabbit's approval is enough to satisfy the rule. With the old fence, marking fix(security): clear all open Dependabot and CodeQL alerts #33 or feat: improvements for PyPI publishing and testing #13 ready after this merged would have armed auto-merge, and they'd have squash-merged the moment CodeRabbit approved, with nobody else in the loop.push, soReleasestill runs. If you want your own PRs armed again, delete theuser.type == 'Bot'line.What
renovate.json. It extends the shared presetgithub>rishitank/.github:python. That preset chains togithub>rishitank/.github(default.json), which sets:config:recommended, semantic commits and the Dependency Dashboard;helpers:pinGitHubActionDigests: GitHub Actions are SHA-pinned, and all Actions bumps come as one grouped PR;security;major-upgradelabel, 7-day minimum release age);ruff,mypyandpip-auditare never automerged (tooling-upgrade);uv.lockmaintenance andrangeStrategy: bump..github/dependabot.yml, so Dependabot stops opening version-update PRs. Dependabot alerts stay on. They're a repository setting (Settings → Code security), not part of this file. Renovate reads those same alerts viavulnerabilityAlerts..github/workflows/auto-merge.yml, but it no longer arms Renovate PRs, Dependabot PRs or user-account PRs.PAT_AUTO_MERGE, on every non-draft, same-repo PR.Why
You decided to move from Dependabot to Renovate. This supersedes #29 (closed) and the stale Actions bumps #30 and #31 (closed). Those come back as one grouped, digest-pinned Renovate PR.
How it was verified
npx renovate-config-validator renovate.jsonreturns "Config validated successfully".python.jsonanddefault.jsoninrishitank/.githubwere read at3978d59.auto-merge.yml: only theif:expression and its comment change.renovate.jsonidentical to this branch's, no conflict markers, valid JSON).Risks
pyproject.tomluses>=ranges. WithrangeStrategy: bump, Renovate will raise those lower bounds as it updates.What Rishi must do
pull_requestevents, GitHub runs the workflow from the PR's merge commit, so this PR is governed by its own fenced copy, which skips user-account PRs.🤖 Generated with Claude Code
https://claude.ai/code/session_0171XyyqTUH64gAi1AcerwjN