Skip to content

fix: publish workflow recognizes custom PR merge subjects#121

Merged
hugo-valle merged 3 commits into
mainfrom
develop
Jun 24, 2026
Merged

fix: publish workflow recognizes custom PR merge subjects#121
hugo-valle merged 3 commits into
mainfrom
develop

Conversation

@hugo-valle

Copy link
Copy Markdown
Owner

Summary

  • The check-release step in publish.yml only matched two commit message patterns:
    • Merge pull request #N from org/release/vX.Y.Z (default GitHub merge)
    • Merge branch 'release/vX.Y.Z' (local merge)
  • When a custom --subject is passed to gh pr merge (e.g. "Release v0.4.2"), neither pattern matched and publication was skipped with ❌ Not a release or hotfix branch merge.

Fix

Added two additional patterns to the check:

  • ^Release v?X — matches custom subjects like "Release v0.4.2"
  • ^Hotfix v?X — matches custom subjects like "Hotfix v0.4.2"

When no release/ path appears in the commit message, the branch name now falls back to reading the version from pyproject.toml (e.g. release/v0.4.2).

Test plan

  • Merge a release branch with the default GitHub subject → still triggers publish
  • Merge a release branch with a custom subject like "Release v0.4.X" → now triggers publish

🤖 Generated with Claude Code

Hugo Valle and others added 3 commits June 23, 2026 21:09
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The check-release step only matched "Merge pull request ... from .../release/"
and "Merge branch 'release/..." patterns. A custom --subject like "Release v0.4.2"
was not detected. Add a third pattern that matches commit messages starting with
"Release vX" or "Hotfix vX", and fall back to pyproject.toml for the branch name
when no release/ path appears in the commit message.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hugo-valle hugo-valle merged commit bf395ae into main Jun 24, 2026
25 of 26 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.

1 participant