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 rebase stuff #23432

Merged
merged 2 commits into from
Aug 25, 2017
Merged

Doc rebase stuff #23432

merged 2 commits into from
Aug 25, 2017

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Aug 24, 2017

No description provided.

@kshyatt kshyatt added 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 18:19
should be done "quietly". Used for interoperability. The default is `1`.
* `inmemory`: start an in-memory rebase. Callers working on the rebase can go through its
steps and commit any changes, but cannot rewind HEAD or update the repository. The
[`workdir`](@ref) will not be modified. Only present on libgit2 versions newer than 0.24.0.
Copy link
Member

@ararslan ararslan Aug 24, 2017

Choose a reason for hiding this comment

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

"Newer than" implies to me strictly greater than 0.24.0, which suggests that it was introduced in 0.24.1. Is that accurate?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@static if LibGit2.VERSION >= v"0.24.0"

should we change to "as least as new as"?

Copy link
Member

Choose a reason for hiding this comment

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

That or "version 0.24.0 or newer."

picked "newer than or equal to" for consistency with existing docstrings
@kshyatt kshyatt merged commit f82fc0f into master Aug 25, 2017
@kshyatt kshyatt deleted the ksh/moregitdocs branch August 25, 2017 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants