Skip to content

Don't always use API to check for branch existence#25

Merged
labkey-tchad merged 5 commits intodevelopfrom
fb_revParse
Mar 10, 2026
Merged

Don't always use API to check for branch existence#25
labkey-tchad merged 5 commits intodevelopfrom
fb_revParse

Conversation

@labkey-tchad
Copy link
Member

Rationale

A recent branch_release run targeted the wrong branch because a github hiccup. We use hub api calls to check which branch should be targeted, expecting an error response to indicate that a branch doesn't exist.
For some of these calls, it isn't necessary to call the API, we can just use git rev-parse to check for branch/tag existence.

Related Pull Requests

  • N/A

Changes

  • Don't always use API to check for branch existence

Copy link

@labkey-willm labkey-willm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one typo, otherwise lgtm

@labkey-tchad labkey-tchad merged commit 4c4fd53 into develop Mar 10, 2026
4 checks passed
@labkey-tchad labkey-tchad deleted the fb_revParse branch March 10, 2026 21:25
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