Repository navigation
ci(ci): group Dependabot updates and stop linting its own commits - #182
Merged
Merged
Conversation
Switching the ecosystem to uv made Dependabot open one PR per locked dependency, and every one failed Commit Lint: with no Dependabot commit in main's recent history to copy, it guessed "build(deps-dev): Bump" from human commits, and subject-case rejects the capital. Group updates (all actions; ruff and ty; everything else in uv) and pin the "chore(deps)" prefix the existing Dependabot commits already use. Casing still comes from Dependabot's heuristic, so Commit Lint now skips commits authored by Dependabot on PRs Dependabot opened; human commits pushed onto those PRs are still linted. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Description
204d5d3switched Dependabot frompiptouv, and it reachedmain. Dependabot then opened one PR per locked dependency (#175–#180), and every one fails Commit Lint:Why: dependabot-core copies casing from its last commit in the default branch's 100 most recent commits.
mainhas none, so it falls back to a heuristic over human commits, which guessedbuild(deps-dev): Bump. The 18 earlier Dependabot commits on develop were allchore(deps): bump ….The pinned prefix fixes type/scope. It does not fix casing:
capitalize_first_word?in dependabot-core ignores the configured prefix. So the lint skip is what actually unblocks these PRs. It's limited to Dependabot-opened PRs and Dependabot-authored commits, so a human commit pushed onto a Dependabot PR is still linted. Dependabot's commit bodies are a fixed template (Bumps…,updated-dependencies:), so the skip can't hide an accidental major-bump line (#174).Timing: Dependabot reads
dependabot.ymlfrommain, so grouping and the prefix take effect only after the next release. The Commit Lint change applies to PRs intodevelopas soon as this merges.Related Issues
Unblocks #175–#180. #175 also fails Type Check: ty 0.0.86 flags
sorted()over a set that may containNoneinhelpers/_territory_battle.py. That needs a separate code fix.Type of Change
Checklist
developbranchCHANGELOG.mdby handsrc/changes, so no unit tests / pytest / ty runTesting
fix(core): …commit stacked on top → only that commit is linted ✔.subject-case✖ andscope-enum⚠ above.git merge-treeshows clean merges with ci(release): guard prepare-release against accidental major bumps #174 and ci(ci): pin commitlint once and run the same hook in CI #181 (ci(ci): pin commitlint once and run the same hook in CI #181 also editscommitlint.yml; the hunks don't overlap).Merge danger: two-way door. The blast radius is CI only: Dependabot's own commits stop being linted, and Dependabot PRs get batched once this reaches
main.🤖 Generated with Claude Code