Skip to content

ci(commitlint): skip commit-message validation for dependabot PRs#22

Merged
jdwillmsen merged 1 commit into
mainfrom
ci/skip-commitlint-dependabot
Jun 16, 2026
Merged

ci(commitlint): skip commit-message validation for dependabot PRs#22
jdwillmsen merged 1 commit into
mainfrom
ci/skip-commitlint-dependabot

Conversation

@jdwillmsen

Copy link
Copy Markdown
Member

Problem

Every Dependabot PR fails CI at Validate Commit Messages — Dependabot's commit body embeds release notes/compare links with lines >100 chars, tripping body-max-line-length (inherited from @commitlint/config-conventional). This blocked the apps PR queue (e.g. #19/#20/#21 Angular bumps).

Fix

Gate the commitlint step on github.actor != 'dependabot[bot]'. Human PRs are still fully validated; bot PRs skip the message lint (their messages are bot-generated and not under our control).

Not this

🤖 Generated with Claude Code

Dependabot commit bodies embed release notes / compare links whose lines
exceed 100 chars, tripping commitlint body-max-line-length (inherited from
@commitlint/config-conventional) and failing the Validate Commit Messages
step on every dependency PR — blocking the apps PR queue. Gate the step on
github.actor != 'dependabot[bot]' so human PRs are still validated.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Assisted-by: Claude:Opus-4.8 [claude-code]
@nx-cloud

nx-cloud Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit 9ef4b3b

Command Status Duration Result
nx affected -t build ✅ Succeeded <1s View ↗
nx affected -t lint test ✅ Succeeded <1s View ↗
nx-cloud record -- nx format:check ✅ Succeeded <1s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-06-16 02:34:12 UTC

@jdwillmsen jdwillmsen merged commit e74f6e8 into main Jun 16, 2026
2 checks passed
@jdwillmsen jdwillmsen deleted the ci/skip-commitlint-dependabot branch June 16, 2026 02:35
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