Skip to content

fix: check out PR head commit in conform workflow - #143

Merged
rdavid merged 1 commit into
masterfrom
fix/conform-pr-head-ref
Aug 25, 2026
Merged

fix: check out PR head commit in conform workflow#143
rdavid merged 1 commit into
masterfrom
fix/conform-pr-head-ref

Conversation

@rdavid

@rdavid rdavid commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • The conform workflow's checkout used GitHub's default pull_request ref, which is the synthetic refs/pull/*/merge commit
  • That commit's auto-generated message ("Merge into ") is not a conventional commit, so conform enforce failed on every PR regardless of the actual commit messages
  • Pin checkout to github.event.pull_request.head.sha (falling back to github.sha on push/dispatch) so only real commits are checked

Test plan

  • Confirm the conform check passes on this PR

GitHub's default checkout ref for pull_request events is the synthetic
refs/pull/*/merge commit, whose own auto-generated message ("Merge
<sha> into <sha>") fails the conventional-commit check on every PR.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c020751a-c669-4c07-a7ea-1ff68c944943


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@rdavid
rdavid merged commit 1b1e57c into master Aug 25, 2026
11 checks passed
rdavid added a commit that referenced this pull request Aug 25, 2026
GitHub's default checkout ref for pull_request events is the synthetic
refs/pull/*/merge commit, whose own auto-generated message ("Merge
<sha> into <sha>") fails the conventional-commit check on every PR.
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.

1 participant