Skip to content

Conversation

@ZolotukhinM
Copy link
Collaborator

@ZolotukhinM ZolotukhinM commented Oct 12, 2024

Stacked PRs:


Don't attempt to delete remote branch after a PR has landed.

GitHub deletes the remote branch automatically when the PR is merged, so
we don't to explicitly do that. Trying to delete it currently results in
an error like this:

error: unable to delete 'ZolotukhinM/stack/1': remote ref does not exist
error: failed to push some refs to '...'

The code for deleting the branch has been sitting there for a long time,
but until a fix in #27 it used incorrect branch names.

GitHub deletes the remote branch automatically when the PR is merged, so
we don't to explicitly do that. Trying to delete it currently results in
an error like this:
```
error: unable to delete 'ZolotukhinM/stack/1': remote ref does not exist
error: failed to push some refs to '...'
```
The code for deleting the branch has been sitting there for a long time,
but until a fix in #27 it used incorrect branch names.

stack-info: PR: #42, branch: ZolotukhinM/stack/1
@ZolotukhinM ZolotukhinM merged commit 7cbd8ab into main Oct 12, 2024
@ZolotukhinM ZolotukhinM deleted the ZolotukhinM/stack/1 branch October 12, 2024 04:15
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.

2 participants