-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
reset changes for merge #1013
reset changes for merge #1013
Conversation
@@ -88,6 +90,11 @@ int git_reset( | |||
goto cleanup; | |||
} | |||
|
|||
if (reset_type == GIT_RESET_SOFT && (git_repository_state(repo) == GIT_REPOSITORY_STATE_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.
How about introducing a new GIT_EUNMERGED
error code and return it here?
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.
Yep.
On Oct 23, 2012, at 4:02 PM, nulltoken notifications@github.com wrote:
In src/reset.c:
@@ -88,6 +90,11 @@ int git_reset(
goto cleanup;
}
- if (reset_type == GIT_RESET_SOFT && (git_repository_state(repo) == GIT_REPOSITORY_STATE_MERGE)) {
How about introducing a new GIT_EUNMERGED error code and return it here?—
Reply to this email directly or view it on GitHub.
So far so good... would it make sense to add the other states (rebase, am, bisect) now too? Thoughts on how to expose the "interactive head" during a rebase? libgit2/libgit2sharp#128 (comment) |
Rebased on development. @vmg, is this ready for merge or are there additional changes? |
Looks great. One of the travis builds has stalled but the other are clean. |
reset changes for merge
Includes previous PR plus https://github.com/carlosmn/libgit2/blob/e849aa83003d209c53513c69c3c56a8fbfd38f1f/tests-clar/repo/state.c