Skip to content

fix: open the PR for a bump branch that has none - #486

Open
drptbl wants to merge 1 commit into
dappnode:masterfrom
drptbl:fix/bump-upstream-orphan-branch
Open

drptbl wants to merge 1 commit into
dappnode:masterfrom
drptbl:fix/bump-upstream-orphan-branch

Conversation

@drptbl

@drptbl drptbl commented Nov 10, 2026

Copy link
Copy Markdown

bump-upstream assumes an existing bump branch already has a PR. When the token isn't allowed to create pull requests, the branch gets pushed, PR creation fails, and every later run logs Branch ... already exists and passes. That's how https://github.com/xAlisher/logos-node-dappnode stayed on 0.3.1 while 0.3.2 and 0.3.4 shipped (https://github.com/xAlisher/logos-node-dappnode/actions/runs/41197630281).

Now a bump branch that exists on the remote with no PR in any state gets its PR opened. A branch whose PR was closed is still skipped, closing it was a decision.

bump-upstream assumed an existing bump branch already had a PR, so a branch whose PR creation was refused (e.g. GitHub Actions not allowed to create pull requests) made every later run skip that release.
@drptbl
drptbl requested a review from a team as a code owner November 10, 2026 12:35
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