Adopt the fleet's dependabot config and auto-merge gate - #10
Merged
Merged
Conversation
allocator had neither, while roughly thirty-five repos in the fleet have both. Copied unchanged from calibre so there is one policy rather than a second dialect of it. What it does. dependabot.yml watches the uv and github-actions ecosystems weekly, with a seven-day cooldown so a release has to survive a week in the wild before it is proposed -- security updates skip the cooldown. Updates are grouped into minor-and-patch and security sets. dependabot-auto-merge.yml then arms `gh pr merge --auto` for the eligible ones: GitHub-Actions updates, the minor-and-patch groups, and patch/minor Python bumps. Python *majors* are deliberately not eligible and get a comment saying so. `--auto` is the part that matters -- it merges only once the required checks pass, so this is auto-merge behind the tests rather than instead of them. This is only worth having now because of the two changes before it. Until uv.lock was committed there was nothing for dependabot to bump, and until CI installed with `uv sync --locked` a dependency bump was not validated against anything. The four dependabot PRs already open are lock-only transitive updates, and all four are green across the full 8-to-10 check matrix -- which is the evidence that the loop now works end to end. Not adopting preen here: allocator has no .copier-answers.yml and adopting the py-canon template rewrites [tool.*] blocks and workflow shims, which is a separate decision from dependency hygiene. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
allocator had neither, while ~35 repos in the fleet have both. Copied unchanged from calibre so there is one policy rather than a second dialect of it.
What it does
dependabot.ymlwatches the uv and github-actions ecosystems weekly, with a seven-day cooldown so a release has to survive a week in the wild before it's proposed — security updates skip the cooldown. Updates are grouped into minor-and-patch and security sets.dependabot-auto-merge.ymlthen armsgh pr merge --autofor the eligible ones:--autois the part that matters: it merges only once the required checks pass, so this is auto-merge behind the tests rather than instead of them.Why it's worth having now
Only because of the two changes before it. Until
uv.lockwas committed there was nothing for dependabot to bump, and until CI installed withuv sync --lockeda dependency bump wasn't validated against anything.The four dependabot PRs already open (#6–#9) are lock-only transitive updates, and all four are green across the full 8–10 check matrix — which is the evidence that the loop now works end to end. Under this policy #7 (pillow minor) and #8 (pyasn1 patch) would auto-merge; #6 (cryptography major) and #9 (black major) would be left for review, which is the right call.
Not adopting preen here: allocator has no
.copier-answers.yml, and adopting the py-canon template rewrites[tool.*]blocks and workflow shims. That's a separate decision from dependency hygiene.🤖 Generated with Claude Code