Skip to content

Checkout (restore) instead of reset branch head #35298

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

Closed
wants to merge 1 commit into from

Conversation

jablko
Copy link
Contributor

@jablko jablko commented Nov 22, 2019

Checkout (restore) is sufficient, since the goal is to update the working tree, not the branch head.

@jablko jablko force-pushed the patch-28 branch 2 times, most recently from 0a50057 to 56223a0 Compare November 26, 2019 15:29
@jablko jablko force-pushed the patch-28 branch 3 times, most recently from b8d6db6 to 82385de Compare November 29, 2019 22:25
@jablko jablko force-pushed the patch-28 branch 3 times, most recently from 23d7418 to d6abd1d Compare January 25, 2020 20:23
@sandersn sandersn added the Housekeeping Housekeeping PRs label Feb 1, 2020
@jablko jablko force-pushed the patch-28 branch 4 times, most recently from a93aa90 to 6c8fead Compare February 6, 2020 14:56
Copy link
Member

@sandersn sandersn left a comment

Choose a reason for hiding this comment

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

Works for me, but can you explain the advantage of checkout :/ over reset HEAD --hard?

@jablko jablko force-pushed the patch-28 branch 6 times, most recently from 6b080fd to 06dec19 Compare February 13, 2020 15:43
@sandersn
Copy link
Member

This PR hasn't seen any activity for quite a while, so I'm going to close it to keep the number of open PRs manageable. Feel free to open a fresh PR or continue the discussion here.

@sandersn sandersn closed this Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Housekeeping Housekeeping PRs
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants