Skip to content

fix(release): make mergeability poll executable - #314

Merged
0xPlayerOne merged 2 commits into
stagingfrom
fix/release-auto-merge-race
Aug 1, 2026
Merged

fix(release): make mergeability poll executable#314
0xPlayerOne merged 2 commits into
stagingfrom
fix/release-auto-merge-race

Conversation

@0xPlayerOne

Copy link
Copy Markdown
Owner

Summary

  • correct the gh pr view jq invocation in the mergeability guard
  • add a regression assertion so the invalid two-argument form cannot return UNKNOWN forever

The v0.34.0 reusable release workflow accepted the new guard in PR #311, but the first live release run exposed this syntax defect before the bounded timeout. This follow-up is intentionally isolated and keeps consumers on their existing pins until a deliberate v0.34.1 migration.

Validation

  • npm test (78 passed)
  • npm run build
  • npm run type-check
  • actionlint (errors none; existing SC2086 info diagnostics unchanged)

@0xPlayerOne
0xPlayerOne merged commit 16dd1ab into staging Aug 1, 2026
17 checks passed
@0xPlayerOne
0xPlayerOne deleted the fix/release-auto-merge-race branch August 1, 2026 22:20
0xPlayerOne added a commit that referenced this pull request Aug 1, 2026
* fix(release): use valid gh jq expression for mergeability poll

* test(release): guard gh jq mergeability syntax
@kilo-code-bot

kilo-code-bot Bot commented Aug 1, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • .github/workflows/release.yml - Removed erroneous -r flag from --jq argument; @tsv already produces raw output
  • test/cli.test.mjs - Added regression assertion to prevent re-introduction of broken --jq -r form

Reviewed by step-3.7-flash · Input: 85.1K · Output: 4.2K · Cached: 273.8K

0xPlayerOne added a commit that referenced this pull request Aug 1, 2026
* fix(release): use valid gh jq expression for mergeability poll

* test(release): guard gh jq mergeability syntax
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