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

bump-formula-pr: return to starting branch #760

Merged

Conversation

jasonkarns
Copy link
Contributor

After branching, bumping, pushing, and pr-ing; return to whatever branch
was originally checked out.

In most cases, I'd imagine users to want to continue receiving tap
updates from master. However, after using bump-formula-pr, the tap in
which the formula was bumped is left on the working branch that was
doing the bumping and pull-request. After opening the PR, we should
return to whatever branch the user originally had checked out – most
likely master. (But git allows us to just say "previous branch" by using
-)

After branching, bumping, pushing, and pr-ing; return to whatever branch
was originally checked out.

In most cases, I'd imagine users to want to continue receiving tap
updates from master. However, after using bump-formula-pr, the tap in
which the formula was bumped is left on the working branch that was
doing the bumping and pull-request. After opening the PR, we should
return to whatever branch the user originally had checked out – most
likely master. (But git allows us to just say "previous branch" by using
`-`)
@UniqMartin
Copy link
Contributor

This makes a lot of sense to me, but would prefer @ilovezfs as the most prominent user of this developer command to chime in and comment on this, too.

@ilovezfs
Copy link
Contributor

e9da9f0d (ilovezfs         2016-07-06 05:00:45 -0700 218)         safe_system "git", "checkout", "-"

@UniqMartin UniqMartin merged commit bb4d168 into Homebrew:master Aug 20, 2016
@UniqMartin
Copy link
Contributor

Thanks for your contribution and this tweak to the bump-formula-pr command, @jasonkarns! 🎉

@Homebrew Homebrew locked and limited conversation to collaborators May 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants