-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
More docs for merge! #23495
More docs for merge! #23495
Conversation
base/libgit2/merge.jl
Outdated
head_ann = LibGit2.GitAnnotated(repo, "master") | ||
|
||
# merge the branch in | ||
LibGit2.merge!(repo, [upst_ann]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this supposed to reference head_ann
as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
derp no i c/ped this from the tests
@@ -111,6 +111,29 @@ function ffmerge!(repo::GitRepo, ann::GitAnnotated) | |||
end | |||
|
|||
# Merge changes into current head |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment seem a bit superfluous now that we have a complete docstring?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think they aren't harming anyone by hanging around. In general I'd rather err on the side of having more context rather than less...
@@ -126,6 +149,40 @@ end | |||
|
|||
# Internal implementation of merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same with these
Forgot to add
GitAnnotated
to the devdocs, fixed that too.