Skip to content

chore(deps-dev): bump @commitlint/cli and config-conventional to v20#242

Merged
AbongileBoja merged 1 commit intodevelopfrom
chore/deps/commitlint-v20
Apr 28, 2026
Merged

chore(deps-dev): bump @commitlint/cli and config-conventional to v20#242
AbongileBoja merged 1 commit intodevelopfrom
chore/deps/commitlint-v20

Conversation

@AbongileBoja
Copy link
Copy Markdown
Owner

Summary

Replaces dependabot PR #209 with a coupled bump that keeps @commitlint/cli and @commitlint/config-conventional on the same major.

Why not just merge #209

Dependabot only bumped the preset (config-conventional) to v20 while leaving the cli on v19. That split the major version between linter and preset and produced a lockfile with two copies of @commitlint/types (v19 for cli, v20 for preset) installed side-by-side. Even though npm tolerates this, it is the kind of drift that causes silent rule/types mismatches in the future. Lockstep bumps are the enterprise-grade answer.

What changed in v20

Single breaking change: body-max-line-length now ignores lines that contain URLs (commitlint #4486). This is a relaxation — human commits that paste long links no longer get flagged. No impact on the dependabot exemption (#241) since the predicate fires before any rule runs.

Test plan

Local validation with the new toolchain (@commitlint/cli@20.5.2):

Closes

Bumps both `@commitlint/cli` and `@commitlint/config-conventional` from
v19 to v20 in lockstep so the linter and preset stay on the same major.
Dependabot's PR (#209) only bumped the preset, which split the major
version between cli (v19) and preset (v20) and forced npm to install
two copies of @commitlint/types side-by-side.

v20.0.0 has a single breaking change (#4486): body-max-line-length now
ignores lines that contain URLs. This is a relaxation, not a regression,
and is welcome behaviour for human commits that paste links.

Validated locally with the new toolchain:
- Real grouped dependabot commit (#211 head): passes (ignored predicate fires).
- 200-char human body line: still rejected with body-max-line-length.
- Long URL in body line: passes (v20 URL-skip).
- Normal short human commit: passes.

Closes the version-drift concern flagged on #209.
@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Complexity Health
QuerySpec.Analyzers 93% 84% 133
QuerySpec.Analyzers.CodeFixes 90% 70% 50
QuerySpec.Core 93% 84% 959
QuerySpec.DependencyInjection 92% 75% 43
QuerySpec.EFCore 86% 77% 303
Summary 91% (2118 / 2316) 81% (930 / 1143) 1488

@AbongileBoja AbongileBoja merged commit eaea58d into develop Apr 28, 2026
12 checks passed
@AbongileBoja AbongileBoja deleted the chore/deps/commitlint-v20 branch April 28, 2026 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant