This repository was archived by the owner on Sep 11, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 534
ResolveRevision doesn't resolve ambigious revs #823
Labels
Comments
Seems like it should be supported through #706, I'll take another look later this week. |
@antham can you please take a look? |
yes |
In this case you get a warning like this when you do a rev-parse
so what I've done in this case is to return an error. |
novas0x2a
added a commit
to novas0x2a/go-git
that referenced
this issue
May 14, 2019
1) lookups on an annotated tag oid now work 2) there was a lot of complexity around detection of ambiguity, but unlike git, ambiguous refs are rejected (which causes bugs like src-d#823). The new code matches rev-parse's behavior (prefer the OID), though there is no warning path to report the same warning.
novas0x2a
added a commit
to novas0x2a/go-git
that referenced
this issue
May 14, 2019
1) lookups on an annotated tag oid now work 2) there was a lot of complexity around detection of ambiguity, but unlike git, ambiguous refs are rejected (which causes bugs like src-d#823). The new code matches rev-parse's behavior (prefer the OID), though there is no warning path to report the same warning. Signed-off-by: Mike Lundy <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
When a branch has the same name as the SHA of the commit it depends on,
ResolveRevision()
doesn't correctly resolve it.I'm not sure how users get in this situation, but to reproduce:
The text was updated successfully, but these errors were encountered: