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

Checkout a PR branch without resetting anything #518

Closed
DamienCassou opened this issue Sep 14, 2022 Discussed in #511 · 3 comments
Closed

Checkout a PR branch without resetting anything #518

DamienCassou opened this issue Sep 14, 2022 Discussed in #511 · 3 comments

Comments

@DamienCassou
Copy link
Contributor

Discussed in #511

Originally posted by DamienCassou August 19, 2022
I would like to be able to checkout the already existing local branch corresponding to the PR at point in magit-status. Currently, b f (M-x forge-checkout-pullreq) also resets the branch and I don't want that.

Said differently, I would like the same as b b (M-x magit-checkout) but with a list of PRs (and the PR at point by default) instead of a list of branches.

DamienCassou added a commit to DamienCassou/forge that referenced this issue Sep 14, 2022
Closes magit#518.
DamienCassou added a commit to DamienCassou/forge that referenced this issue Oct 24, 2022
Closes magit#518.
@sellout
Copy link

sellout commented Nov 29, 2022

Just for clarity, would this add a fourth option to

A branch named "my-fix" already exists.

⋮

  [r]eset existing "my-fix" branch
  [c]reate new "pr-1234" branch instead
  [a]bort

something like [u]se existing "my-fix" branch?

Because that is a feature I would love.

@DamienCassou
Copy link
Contributor Author

Just for clarity, would this add a fourth option to

The PR I sent avoids this choice and directly checks out the existing branch.

@tarsius tarsius closed this as completed Dec 8, 2022
@DamienCassou
Copy link
Contributor Author

Thank you very much @tarsius!

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 a pull request may close this issue.

3 participants