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

Add 'status' command #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jrdavid
Copy link

@jrdavid jrdavid commented Sep 30, 2013

Hello,

Here's an implementation for a 'status' command, as discussed. Let me know what you think.

Cheers!
JR

@mhagger
Copy link
Owner

mhagger commented Oct 3, 2013

Cool, thanks for working on this! It will be a nice feature.

I belatedly realized that I was working on a topic branch that could help you with your "status" command (I just merged the topic branch to master): I added fields "tip1" and "tip2" to MergeState. These record the names of the branches that are being merged, based on the user's arguments to "git imerge start", because I wanted to add that info to the suggested commit message when simplifying to a merge commit. But you could use these fields instead of your get_refname() method, simplifying the code and potentially giving an answer that is more relevant to the user. (Instead of computing and using ref1 and ref2, simply use merge_state.tip1 and merge_state.tip2.)

I also think that the "status" command should work even if a conflicting merge is not in progress. For example, if the user has just run git commit to commit a conflict resolution, or has just run git imerge record to record the last conflict resolution without setting up the next one, then git imerge status could nevertheless say something useful about the current situation rather than just spitting out an error message. It might even suggest the next command that the user might want to run.

Finally, it looks like the implementation of "status" could share some code with request_user_merge().

Altogether very nice. I look forward to the next iteration. But please be patient; I will have spotty internet for the next week or so and might not be able to reply promptly.

@mhagger mhagger mentioned this pull request Oct 30, 2013
@mhagger
Copy link
Owner

mhagger commented Oct 30, 2013

@jrdavid, are you working on a new version of your patch? If so, is there anything I can help with?

@jrdavid
Copy link
Author

jrdavid commented Oct 30, 2013

I'm not currently working on it, no. I still want to do it but I've just been swamped the last few weeks. You've helped quite a bit already. I think I'm good for now, thanks. I just don't know when I'll be able to get around to it.

@nafg
Copy link

nafg commented Apr 30, 2015

+10

@abitrolly
Copy link
Contributor

Any reason to keep this open?

@jrdavid
Copy link
Author

jrdavid commented Jun 25, 2020

No.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants