Skip to content
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

homebrew_formula_pullrequest: unique branch name #1062

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

scpeters
Copy link
Contributor

@scpeters scpeters commented Nov 9, 2023

The homebrew formula pull requests sometimes are triggered twice, but the second pull request is not created due to overlapping branch names. This gives each pull request a unique branch name so that duplicate pull requests will be created, and the maintainer can figure out which has the correct tarball SHA.

Fixes #1020.

@scpeters scpeters requested a review from j-rivero as a code owner November 9, 2023 18:59
@scpeters
Copy link
Contributor Author

scpeters commented Nov 9, 2023

@scpeters
Copy link
Contributor Author

scpeters commented Nov 9, 2023

the first pull request (osrf/homebrew-simulation#2491) has the wrong sha256

the new pull request (osrf/homebrew-simulation#2492) made with this branch has the right sha256

@scpeters
Copy link
Contributor Author

scpeters commented Nov 9, 2023

it worked!

@j-rivero j-rivero merged commit badf2e9 into master Nov 17, 2023
1 check passed
@j-rivero j-rivero deleted the scpeters/brew_pr_unique_branch_name branch November 17, 2023 18:47
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.

Running release.py two times invalidates the tarball SHA256 in homebrew pull request
2 participants