ci(commitlint): ignore dependabot bump commits#51
Merged
Conversation
kj-podonos
force-pushed
the
ci/commitlint-ignore-dependabot
branch
from
fdc565f to
c03e210
Compare
kj-podonos
force-pushed
the
ci/commitlint-ignore-dependabot
branch
from
c03e210 to
aee71c7
Compare
Migrate .commitlintrc.json to commitlint.config.cjs so we can use a function-based `ignores` rule (not expressible in JSON). Ignore Dependabot's default `Bump <dep> from <x> to <y>` subject, which has no Conventional Commits type prefix and is produced for indirect (transitive) dependency updates even though .github/dependabot.yml sets a `chore` prefix. That mismatch failed the commitlint check on PR #50 (cryptography 48.0.1). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
kj-podonos
force-pushed
the
ci/commitlint-ignore-dependabot
branch
from
aee71c7 to
45bf8bf
Compare
3 tasks
kj-podonos
added a commit
that referenced
this pull request
Jun 18, 2026
## Summary Bumps the transitive dependency **cryptography** `48.0.0 → 49.0.0` in `uv.lock`. Supersedes the stale dependabot PR #50 (targeted 48.0.1; its branch was deleted after the commitlint fix #51 landed). 49.0.0 is the current latest. ## Compatibility - **Transitive, dev-only, linux-only:** pulled in via `twine → keyring → secretstorage`; `secretstorage` carries `sys_platform == 'linux'`. **Zero direct usage** in `src/` or `tests/`. - `secretstorage` depends on `cryptography` with no upper bound → 49.0.0 satisfies it (uv resolved 72 pkgs, only cryptography changed). - Lock has `cp39-abi3` wheels → cryptography 49.0.0 supports **Python ≥ 3.9**; repo requires `>= 3.10`. manylinux_2_28/2_34 + musllinux wheels present. - Lock-only change; no source touched. ## Test plan - [ ] CI matrix green (Python 3.10–3.14 × ubuntu/macos/windows) - [ ] `fresh-venv-smoke` green (real install) - [ ] `lint` green (conventional commit) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.8 (1M context) <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
1 participant
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.
What
Migrate
.commitlintrc.json→commitlint.config.cjsand add a function-basedignoresrule that skips Dependabot's defaultBump <dep> from <x> to <y>commit subject.
Why
The
lint(commitlint) check fails on dependabot PRs whose commit subject lacksa Conventional Commits
type:prefix —type-empty+subject-empty..github/dependabot.ymlalready setscommit-message.prefix: chore, and it worksfor github-actions updates (PRs #45–49 are
chore(deps): bump …). But it isnot applied to indirect/transitive pip deps, so PR #50
(
Bump cryptography from 48.0.0 to 48.0.1) failed commitlint while all 18 testmatrix jobs passed. This is a recurring gap for future transitive bumps.
A function-based
ignorespredicate can't live in JSON, hence the.cjsmigration.extends(@commitlint/config-conventional) and thebody-max-line-lengthruleare preserved unchanged.
Verification
Bump … from … to …while still lintingnormal/
chore(deps):commits (verified locally via node).commitlintjob is the real gate on this PR.Unblocks re-driving #50 (cryptography 48.0.1).
🤖 Generated with Claude Code