Skip to content

fix(workflow): check exit code, not emptiness, for parent-issue lookup - #371

Merged
arutsh merged 1 commit into
mainfrom
Platform/fix/Issue-370/close-linked-issues-parent-lookup
Sep 30, 2026
Merged

arutsh merged 1 commit into
mainfrom
Platform/fix/Issue-370/close-linked-issues-parent-lookup

Conversation

@arutsh

@arutsh arutsh commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

Test plan

🤖 Generated with Claude Code

#370)

gh api --jq writes the raw error body to stdout on a failed request, not
just stderr, so `parent=$(gh api ... || echo "")` still captured that
body once the request failed. It then got used as a URL path segment in
the next gh api call and broke. Confirmed live: PR #369's own merge run
closed #368, then crashed exactly this way since #368 has no parent.

Check the command's own exit status instead of trusting the string.

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

vercel Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
grant-flow Ready Ready Preview Sep 30, 2026 1:40pm UTC

@arutsh
arutsh merged commit 0674851 into main Sep 30, 2026
30 checks passed
@arutsh
arutsh deleted the Platform/fix/Issue-370/close-linked-issues-parent-lookup branch October 10, 2026 09:00

This branch was successfully deployed

1 active deployment
Preview — 834b2d49 Deployed Sep 30, 2026 by vercel[bot]
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.

Close linked sub-issues automatically on PR merge (squash-safe)

1 participant