Skip to content

fix: auto-release PR detection (parse from commit message)#9

Merged
szibis merged 1 commit intomainfrom
fix/auto-release-v2
Apr 6, 2026
Merged

fix: auto-release PR detection (parse from commit message)#9
szibis merged 1 commit intomainfrom
fix/auto-release-v2

Conversation

@szibis
Copy link
Copy Markdown
Owner

@szibis szibis commented Apr 6, 2026

Fixes race condition where API hasn't indexed PR association yet. Parses (#N) from squash merge commit message.

…ition)

The gh api commits/<sha>/pulls endpoint fails when called immediately after
a squash merge — GitHub hasn't indexed the PR association yet.

Fix: Parse PR number from squash merge commit message "(#N)" first,
fall back to API with 5s delay if not found.
@szibis szibis added the release:minor Triggers minor version bump (0.X.0) — new features label Apr 6, 2026
@szibis szibis merged commit af6eebc into main Apr 6, 2026
4 checks passed
szibis added a commit that referenced this pull request Apr 6, 2026
Shell was interpreting parentheses in squash merge commit messages
like "feat: something (#9)" as subshell commands. Moving COMMIT_MSG
to step-level env: block passes the value safely. Also simplified
the grep pattern for more robust PR number extraction.
szibis added a commit that referenced this pull request Apr 6, 2026
Shell was interpreting parentheses in squash merge commit messages
like "feat: something (#9)" as subshell commands. Moving COMMIT_MSG
to step-level env: block passes the value safely. Also simplified
the grep pattern for more robust PR number extraction.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:minor Triggers minor version bump (0.X.0) — new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant