Skip to content

ci: replace wagoid commitlint action with locally installed CLI#124

Merged
wadackel merged 1 commit into
mainfrom
chore/commitlint-1
May 16, 2026
Merged

ci: replace wagoid commitlint action with locally installed CLI#124
wadackel merged 1 commit into
mainfrom
chore/commitlint-1

Conversation

@wadackel
Copy link
Copy Markdown
Collaborator

@wadackel wadackel commented May 16, 2026

Summary

  • Drop the third-party wagoid/commitlint-github-action from .github/workflows/commitlint.yaml and run the already installed @commitlint/cli directly, mirroring the existing ci.yaml pattern (actions/checkout + ./.github/actions/setup + pnpm <step>) and the local lefthook usage (pnpm exec commitlint --edit {1}).
  • The commit range is derived from pull_request.base.sha / head.sha, passed via env: to avoid inline ${{ }} expansion in run: (defense-in-depth against expression injection).
  • fetch-depth: 0 is preserved so the base SHA is reachable. No changes to triggers, permissions, job name, or runner.

🤖 Generated with Claude Code

Run commitlint via `pnpm exec` after `./.github/actions/setup`, mirroring
the existing ci.yaml pattern (checkout + setup composite + pnpm <step>)
and the local lefthook usage (`pnpm exec commitlint --edit {1}`). This
drops the third-party `wagoid/commitlint-github-action` dependency and
unifies CI / local execution on the already installed `@commitlint/cli`.

The base/head SHAs are passed via `env:` instead of inline `${{ }}`
expansion as a defense-in-depth measure against expression injection.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@wadackel wadackel merged commit b2fcd88 into main May 16, 2026
7 checks passed
@wadackel wadackel deleted the chore/commitlint-1 branch May 16, 2026 13:53
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