Skip to content

ci: add PR commit message sanity check workflow#10286

Merged
JacobBarthelmeh merged 1 commit intowolfSSL:masterfrom
LinuxJedi:git-action
Apr 23, 2026
Merged

ci: add PR commit message sanity check workflow#10286
JacobBarthelmeh merged 1 commit intowolfSSL:masterfrom
LinuxJedi:git-action

Conversation

@LinuxJedi
Copy link
Copy Markdown
Member

@LinuxJedi LinuxJedi commented Apr 23, 2026

Adds a GitHub Actions workflow that scans every commit in a pull request and fails if any commit message carries a Co-authored-by or Signed-off-by trailer pointing at noreply@anthropic.com.

Copilot AI review requested due to automatic review settings April 23, 2026 05:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new GitHub Actions workflow to enforce commit-message hygiene on pull requests by scanning PR commits for disallowed AI attribution trailers (specifically Co-authored-by: / Signed-off-by: referencing noreply@anthropic.com).

Changes:

  • Introduces a pr-commit-check.yml workflow triggered on pull_request.
  • Fetches full git history and iterates over PR commits to grep commit messages for the disallowed trailers.
  • Fails the workflow with an error annotation if any commit violates the rule.

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

Comment thread .github/workflows/pr-commit-check.yml Outdated
Comment thread .github/workflows/pr-commit-check.yml
Adds a GitHub Actions workflow that scans every commit in a pull
request and fails if any commit message carries a Co-authored-by
or Signed-off-by trailer pointing at noreply@anthropic.com.
@github-actions
Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@LinuxJedi
Copy link
Copy Markdown
Member Author

Jenkins retest this please, unrelated failure

Copy link
Copy Markdown
Contributor

@JacobBarthelmeh JacobBarthelmeh left a comment

Choose a reason for hiding this comment

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

Likely should be expanded to trigger on other co-author AI cases. This looks like a great first test addition.

@JacobBarthelmeh JacobBarthelmeh merged commit 20c1b91 into wolfSSL:master Apr 23, 2026
429 of 431 checks 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.

4 participants