Skip to content

ci: add semantic PR title check - #15

Merged
ClayMav merged 3 commits into
mainfrom
add-semantic-title-check
Apr 24, 2026
Merged

ci: add semantic PR title check#15
ClayMav merged 3 commits into
mainfrom
add-semantic-title-check

Conversation

@james-willis

@james-willis james-willis commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds amannn/action-semantic-pull-request@v5 workflow to enforce conventional commit prefixes on PR titles
  • Adds conventional-pre-commit hook to validate commit messages locally
  • Allowed types: feat, fix, chore, refactor, perf, test, ci, docs, build, style, revert

Test plan

  • Open a PR with a non-conventional title and verify the check fails
  • Run pre-commit install --hook-type commit-msg locally, then try committing with a bad prefix — should be rejected

@james-willis
james-willis requested a review from a team as a code owner April 6, 2026 19:48
@james-willis
james-willis requested review from haizhou-zhao and removed request for a team April 6, 2026 19:48
@ClayMav
ClayMav merged commit 58511d4 into main Apr 24, 2026
2 checks passed
@ClayMav
ClayMav deleted the add-semantic-title-check branch April 24, 2026 19:23
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.

2 participants