Skip to content

Let release.sh start over after a closed bump PR's branch is deleted - #479

Merged
JacobStephens2 merged 1 commit into
mainfrom
fix-release-closed-pr
Oct 7, 2026
Merged

JacobStephens2 merged 1 commit into
mainfrom
fix-release-closed-pr

Conversation

@JacobStephens2

Copy link
Copy Markdown
Owner

Summary

When release.sh finds a bump PR closed without merging, it refuses with "reopen it or delete release- to start over". Deleting the branch didn't help: the next run still found the closed PR as the newest from release-<version> and refused again, so the version couldn't be released without reopening a stale PR.

find_earlier_run now treats a closed PR whose branch is gone from origin as no earlier run, so the script cuts a fresh bump from current main. A closed PR whose branch is still on origin is still refused.

Hit while switching tonight's release from 0.10.1 back to 0.10.0 (#455 closed, release-0.10.0 deleted).

Evidence

  • New test a_bump_pr_closed_without_merging_and_its_branch_deleted_starts_over fails against the old script with the refusal and passes with the fix.
  • cargo test --test release: 46 passed, 0 failed. shellcheck scripts/release.sh clean, cargo fmt --check clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ts55nZd9iVAg8GP3QxHo9Y

The refusal for a bump PR closed without merging says to "delete
release-<version> to start over", but the next run still found the closed
PR and refused again. A closed PR whose branch is gone from origin now
counts as no earlier run.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ts55nZd9iVAg8GP3QxHo9Y
@JacobStephens2
JacobStephens2 merged commit c3073df into main Oct 7, 2026
13 checks passed
@JacobStephens2
JacobStephens2 deleted the fix-release-closed-pr branch October 7, 2026 03:17
@JacobStephens2 JacobStephens2 mentioned this pull request Oct 7, 2026
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