Skip to content

chore(DCP-2422): enforce conventional commits via git hook and CI#321

Merged
script-this merged 2 commits intomainfrom
chore/dcp-2422-automate-cli-release-process-conventional-commits
Feb 26, 2026
Merged

chore(DCP-2422): enforce conventional commits via git hook and CI#321
script-this merged 2 commits intomainfrom
chore/dcp-2422-automate-cli-release-process-conventional-commits

Conversation

@script-this
Copy link
Copy Markdown
Contributor

@script-this script-this commented Feb 26, 2026

Summary

First step toward automating the CLI release process (DCP-2422). Enforces conventional commit format both locally and in CI, which is a prerequisite for automated changelog generation and semantic versioning.

  • Adds a commit-msg git hook (scripts/hooks/commit-msg) that validates commit messages match conventional commit format (<type>(<scope>): <description>)
  • Installs the hook automatically via make install alongside the existing pre-commit hook
  • Adds a GitHub Actions workflow (commitlint.yml) that lints all PR commit messages
  • Updates DEVELOPMENT.md with documentation for both git hooks

Test plan

  • Run make install and verify .git/hooks/commit-msg is installed
  • git commit --allow-empty -m "bad message" — should fail with helpful error
  • git commit --allow-empty -m "chore: test" — should pass
  • git commit --allow-empty -m "feat(DCP-123): test" — should pass
  • Verify CI commitlint workflow runs on this PR

@script-this script-this requested a review from a team as a code owner February 26, 2026 18:57
@prolific-snyk
Copy link
Copy Markdown

prolific-snyk commented Feb 26, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@script-this script-this added team-dct dct-ready-for-review Triggers Slack notification when PR is ready for review labels Feb 26, 2026
@script-this script-this merged commit 6759cd7 into main Feb 26, 2026
9 checks passed
@script-this script-this deleted the chore/dcp-2422-automate-cli-release-process-conventional-commits branch February 26, 2026 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dct-ready-for-review Triggers Slack notification when PR is ready for review team-dct

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants