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

Doc merge_analysis and some enums #23435

Merged
merged 2 commits into from
Aug 27, 2017
Merged

Doc merge_analysis and some enums #23435

merged 2 commits into from
Aug 27, 2017

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Aug 24, 2017

No description provided.

@kshyatt kshyatt added domain:docs This change adds or pertains to documentation libgit2 The libgit2 library or the LibGit2 stdlib module labels Aug 24, 2017
@kshyatt kshyatt requested a review from ararslan August 24, 2017 19:20
@enum(GIT_MERGE_PREFERENCE, MERGE_PREFERENCE_NONE = 0,
MERGE_PREFERENCE_NO_FASTFORWARD = 1,
MERGE_PREFERENCE_FASTFORWARD_ONLY = 2)

""" Result of analysis on merge possibilities.
* `MERGE_ANALYSIS_NONE`: it is not possible to merge the elements of `anns`.
Copy link
Member

Choose a reason for hiding this comment

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

What does anns refer to here?

@@ -93,6 +93,7 @@ Base.LibGit2.lookup_branch
Base.LibGit2.mirror_callback
Base.LibGit2.mirror_cb
Base.LibGit2.message
Base.LibGit2.merge_analysis
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add the newly documented enums in here too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think we have the other enums in this...

Copy link
Member

Choose a reason for hiding this comment

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

Seems like we should though, at least IMO.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See here for a discussion of why we didn't do this

now consistent with the other options
@kshyatt
Copy link
Contributor Author

kshyatt commented Aug 27, 2017

Test fail is in unrelated spawn tests.

@kshyatt kshyatt merged commit e545c65 into master Aug 27, 2017
@kshyatt kshyatt deleted the ksh/docmerge branch August 27, 2017 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:docs This change adds or pertains to documentation libgit2 The libgit2 library or the LibGit2 stdlib module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants