New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(github): Remote branch existence check #23480
fix(github): Remote branch existence check #23480
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise LGTM
…b-remote-branch-existence
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please update PR body to match new behavior
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ups, already done 👍
you need to find the unresolved conversations 🙃 |
🎉 This PR is included in version 36.20.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
The problem is when we have branch
foo/bar
, HEAD request forfoo
will return 200.To fix this, we first make another HEAD request with
/
suffix, and make sure it doesn't return 200 (otherwise, it means, nested refs exist).Context
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: