Skip to content

ci: replace commitizen workflow with PR title check only#487

Merged
lwshang merged 1 commit into
mainfrom
lwshang/ci_remove-check-commit-messages
Apr 30, 2026
Merged

ci: replace commitizen workflow with PR title check only#487
lwshang merged 1 commit into
mainfrom
lwshang/ci_remove-check-commit-messages

Conversation

@lwshang
Copy link
Copy Markdown
Contributor

@lwshang lwshang commented Apr 30, 2026

Summary

  • Remove the check_commit_messages job from the CI workflow
  • Since we use squash merge, only the PR title lands on main — and that's already validated by check_pr_title
  • Rename the workflow file and its name field from commitizen to Check PR title

Test plan

  • Verify the check_pr_title job runs on this PR
  • Confirm no check_commit_messages job appears in the Actions tab

🤖 Generated with Claude Code

Remove the check_commit_messages job since we use squash merge and only
the PR title (already validated) lands on main.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lwshang lwshang marked this pull request as ready for review April 30, 2026 14:42
@lwshang lwshang requested a review from a team as a code owner April 30, 2026 14:42
@lwshang lwshang merged commit a068628 into main Apr 30, 2026
9 checks passed
@lwshang lwshang deleted the lwshang/ci_remove-check-commit-messages branch April 30, 2026 15:33
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.

2 participants