Skip to content

ci: restrict conventional commit types to used set#82

Merged
deviantintegral merged 1 commit into
mainfrom
fix/restrict-conventional-commit-types
Mar 11, 2026
Merged

ci: restrict conventional commit types to used set#82
deviantintegral merged 1 commit into
mainfrom
fix/restrict-conventional-commit-types

Conversation

@deviantintegral
Copy link
Copy Markdown
Owner

Summary

  • Both the pre-commit hook (conventional-pre-commit) and the GitHub workflow (action-semantic-pull-request) accepted any conventional commit type with no restrictions
  • This allowed invalid types like plan: to pass validation
  • Restricts both to the 8 types actually used in this repo: chore, ci, docs, feat, fix, refactor, revert, test

Test plan

  • Verify pre-commit rejects a commit with an invalid type (e.g., plan: test)
  • Verify pre-commit accepts commits with valid types (e.g., fix: test)
  • Verify the GitHub workflow check passes on this PR

🤖 Generated with Claude Code

Both the pre-commit hook and the GitHub workflow for conventional
commits accepted any type string. Restrict both to the 8 types
actually used in this repo: chore, ci, docs, feat, fix, refactor,
revert, test.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@deviantintegral deviantintegral enabled auto-merge (rebase) March 11, 2026 13:01
@deviantintegral deviantintegral merged commit f67d20e into main Mar 11, 2026
5 checks passed
@deviantintegral deviantintegral deleted the fix/restrict-conventional-commit-types branch March 11, 2026 13:01
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