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.

Sign up for GitHub

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

Jump to bottom

lab mr checkout is checking out from the current branch #21

Closed
zaquestion opened this issue Oct 6, 2017 · 0 comments
Closed

lab mr checkout is checking out from the current branch #21

zaquestion opened this issue Oct 6, 2017 · 0 comments
Labels
needs tests

Comments

@zaquestion
Copy link
Owner

ref is getting fetched to head instead of branch name. So the branch doesn't point to the ref and the checkout is incorrect.

$ git mr checkout 587
From gitlab.com/phony
 * branch              refs/merge-requests/111/head -> FETCH_HEAD
   1234567..1234567             -> phony
Switched to branch 'phony'
@zaquestion zaquestion added the bug label Oct 6, 2017
zaquestion added a commit that referenced this issue Oct 6, 2017
@zaquestion
[#21] (mr checkout) properly fetch MR ref to branchname
01145b8
zaquestion added a commit that referenced this issue Oct 6, 2017
@zaquestion
[#21] (mr checkout) properly fetch MR ref to branchname
50ee1ca
zaquestion added a commit that referenced this issue Oct 7, 2017
@zaquestion
Merge pull request #22 from zaquestion/issue21
eae9efa 
[#21] (mr checkout) properly fetch MR ref to branchname
zaquestion added a commit that referenced this issue Nov 12, 2017
@zaquestion
[#21] (mcCheckout) assert resulting branch name and commit
61d4931
@zaquestion zaquestion mentioned this issue Nov 15, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@zaquestion