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

Amend doesn't bring up old commit message. #109

Closed
alok opened this issue Nov 27, 2016 · 12 comments
Closed

Amend doesn't bring up old commit message. #109

alok opened this issue Nov 27, 2016 · 12 comments

Comments

@alok
Copy link

alok commented Nov 27, 2016

I type CA, both inside and outside of the commit box, and it fails to bring up the last commit message (which does exist). Is anyone else having this issue?

@jreybert
Copy link
Owner

@alok CA works well on both branches master and next. Which version of vimagit are you using? What is your current SHA1?

@jreybert
Copy link
Owner

Right, I got it: CA must be called before you opened the commit box. It means that:

S
S
CA
CC

works, but:

S
S
CC
CA
CC

do not work.

Maybe vimagit should detect this pattern and return an error.

@alok
Copy link
Author

alok commented Nov 29, 2016

It could also automatically toggle the commit type when it makes sense to do so.

@jreybert
Copy link
Owner

Hm, it would make sense indeed...

I am entirely satisfied by the actual Commit Amend behaviour, it should show the status of the amended commit. What do you think?

@alok
Copy link
Author

alok commented Jan 24, 2017

It's not that big a deal to me, so do whatever pleases you.

@jreybert
Copy link
Owner

@alok I realize that I forgot something in my last sentence.

My actual question was:
I am NOT entirely satisfied by the actual Commit Amend behavior, it should show the status of the amended commit. What do you think?
To be more clear, there are two kind of amend: amend only the commit message, and amend the commit content (e.g. unstage some file from previous commit).

Would you find this useful?

@jreybert
Copy link
Owner

BTW, I pushed your requested feature in branch dev/issue_109. Could you test it please?

@alok
Copy link
Author

alok commented Mar 6, 2017

it works

@jreybert
Copy link
Owner

jreybert commented Aug 8, 2017

Merged in master and present in release 1.7.2

@jreybert jreybert closed this as completed Aug 8, 2017
@vais
Copy link

vais commented Dec 2, 2019

Hi, @jreybert, did you ever get around to implementing the second kind of amend, i.e. the kind where you can amend commit content? (E.g. unstage files from the commit being amended, review changes, etc.) Because that would be very useful indeed!

I am NOT entirely satisfied by the actual Commit Amend behavior, it should show the status of the amended commit. What do you think?

To be more clear, there are two kind of amend: amend only the commit message, and amend the commit content (e.g. unstage some file from previous commit).

Would you find this useful?

If not, then, for what it's worth at this point in time, yes please :)

@jreybert
Copy link
Owner

jreybert commented Dec 5, 2019

Hi @vais could you open a new request for this feature? I did not have much time to spend on vimagit this last year, but I will try to find some during the christmas holidays.
If you mind, it could be great to share a step by step usage example, to make sure we have the same usage in mind.

@vais
Copy link

vais commented Dec 10, 2019

Sorry for the delay, @jreybert, here's the new issue: #181

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

No branches or pull requests

3 participants