Skip to content

ci(workflow): drop per-commit message lint, keep PR title check#256

Merged
CalvinAllen merged 1 commit intomainfrom
ci/workflow/remove-commit-message-lint
May 8, 2026
Merged

ci(workflow): drop per-commit message lint, keep PR title check#256
CalvinAllen merged 1 commit intomainfrom
ci/workflow/remove-commit-message-lint

Conversation

@CalvinAllen
Copy link
Copy Markdown
Contributor

Summary

  • The per-commit commitlint job in .github/workflows/commit-lint.yml scanned every commit on a PR's branch and blocked merges on header length, even when the PR title (which becomes the squash-merge subject) was already linted separately.
  • Removes the commitlint job. Keeps the lint-pr-title job, which is the source of truth for the conventional-commit format used at merge time.
  • Renames the workflow's display name from "Lint Commit Messages" to "Lint PR Title" so it matches what the workflow actually does. Filename is unchanged to avoid disturbing any existing branch-protection references.

Test plan

  • CI runs on this PR — Lint PR Title should pass; the old Lint Commit Messages check should no longer appear.

The per-commit commitlint job scanned every commit on a PR's branch,
blocking merges on header length even when the PR title (which becomes
the squash-merge subject) was already linted separately. Drop the
`commitlint` job and rename the workflow's display name to reflect that
it now lints only the PR title.
@CalvinAllen CalvinAllen merged commit 9aa77bb into main May 8, 2026
1 check passed
@CalvinAllen CalvinAllen deleted the ci/workflow/remove-commit-message-lint branch May 8, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant