Skip to content

Add CI check for issue references in commit messages#77

Merged
ebrahimebrahim merged 1 commit into
mainfrom
add-precommit-check-for-issue-number
Feb 25, 2026
Merged

Add CI check for issue references in commit messages#77
ebrahimebrahim merged 1 commit into
mainfrom
add-precommit-check-for-issue-number

Conversation

@ebrahimebrahim

@ebrahimebrahim ebrahimebrahim commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

This makes it a CI requirement to reference in issue number in each commit message (in the title or body of the message).

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new GitHub Actions workflow to enforce the documented commit message convention that every commit should reference a GitHub issue. The workflow runs on pull requests and validates that each commit message contains an issue reference or matches an exception pattern.

Changes:

  • Added a new CI workflow that checks all commits in a PR for GitHub issue references
  • Implemented exception patterns for special commit types (Bump, Merge, Revert, fixup!, squash!, amend!)
  • Configured the workflow to fail PRs with commits missing issue references

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/check-commit-messages.yml
Comment thread .github/workflows/check-commit-messages.yml
Comment thread .github/workflows/check-commit-messages.yml
Comment thread .github/workflows/check-commit-messages.yml Outdated
Comment thread .github/workflows/check-commit-messages.yml Outdated
@ebrahimebrahim ebrahimebrahim force-pushed the add-precommit-check-for-issue-number branch from ceaf635 to ccd67e9 Compare February 24, 2026 03:41
@ebrahimebrahim ebrahimebrahim merged commit 3b35b5a into main Feb 25, 2026
1 check passed
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.

3 participants