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

Always use commit message of editing commit #350

Closed
RanceZero opened this issue Jan 18, 2019 · 15 comments
Closed

Always use commit message of editing commit #350

RanceZero opened this issue Jan 18, 2019 · 15 comments
Assignees
Labels
commit dialog An issue related to the commit dialog enhancement New feature or request

Comments

@RanceZero
Copy link

RanceZero commented Jan 18, 2019

Sometimes in my workflow I am thinking of doing a commit and so start writing its commit message, then, decide to edit a previous commit, but when I am about to recommit this previous commit I see the commit message is not of the commit I am editing but from the one I was doing before. I do not know if this is the only occasion where a previous commit message gets stick when it should not, but is an example of some that does.
Maybe my workflow is the wrong one here, but I have countless times found myself commiting edits with a commit message that was not from that commit, and this is troublesome.

@CheyenneWills
Copy link

CheyenneWills commented Jan 19, 2019

I just got bit by this as well.

@RanceZero
Copy link
Author

It always happen when I abort editing a commit and edit other, it is always with the commit message from the one I aborted and not from the one I am editing now.

@wbond wbond added enhancement New feature or request commit dialog An issue related to the commit dialog labels Mar 8, 2019
@CheyenneWills
Copy link

This is a bug. I've been bitten again with this problem.

Editing a commit contents should not result in the commit message being replaced with the message from a different commit.

@CheyenneWills
Copy link

CheyenneWills commented Apr 12, 2019

I can easily reproduce this.

  1. edit commit A's contents
  2. abort rebase
  3. edit commit B's contents
  4. BUG... the commit message is now set to commit A's commit message.

sublime merge Dev Channel, Build 1104

Fails also on Dev Channel, Build 1111

@CheyenneWills
Copy link

Some more info.

It looks like the aborted commit message becomes "sticky", until a commit actually is saved. You can "modify" the commit message and as long as you don't save a commit it follows around for any commit edits until a commit is saved. It also persists across closing the app and starting again

@BenjaminSchaaf
Copy link
Member

I can't reproduce this in dev build 1202

@RanceZero
Copy link
Author

I am not able to reproduce this bug as of build 2039.

@FichteFoll
Copy link

FichteFoll commented Mar 24, 2021

I experienced this exact issue 2 weeks ago on either 2046 or 2048 (can't say for sure since it was the day of 2048's release). Someone else on Discord also noted they ran into it on 2049.

@RanceZero
Copy link
Author

@FichteFoll Should I reopen this issue? Please show a video of issue reproduction or explain me the exact steps you did so that I reproduce it here.

@FichteFoll
Copy link

FichteFoll commented Mar 24, 2021

2021-03-25_00-52-59.mp4

This is on 2048, btw.

@RanceZero
Copy link
Author

@FichteFoll you are correct. Doing the same workflow I reproduced the Issue on build 2049.

@RanceZero RanceZero reopened this Mar 25, 2021
@CheyenneWills
Copy link

Slightly different sequence. Note, the video capture did not pick up the pop-ups. The sequence is:

Selected commit "A"
edit commit -> edit commit contents -- followed by clicking "ok" in the pop-up
click on abort rebase (twice)

Selected commit "B"
edit commit -> edit commit contents -- followed by clicking "ok" in the pop-up
click on commit and continue rebase.

End result, Commit "B"'s commit message is replaced with message commit "A" (the aborted rebase).

2021-03-24.19-01-07.mp4

@dpjohnst dpjohnst self-assigned this Mar 25, 2021
@dpjohnst
Copy link
Member

dpjohnst commented May 5, 2021

Hi all,

This should be resolved as of Sublime Merge Build 2054 (stable).
When you have a spare moment, please confirm that it's been resolved on your end and I can go ahead and close this issue.

Thanks,
- Dylan

@CheyenneWills
Copy link

Hi all,

This should be resolved as of Sublime Merge Build 2054 (stable).
When you have a spare moment, please confirm that it's been resolved on your end and I can go ahead and close this issue.

Thanks,

  • Dylan

I was unable to reproduce the problem by following the steps I listed above (using sublime merge build 2052).

So I believe that the issue has been corrected.

Thanks
/Cheyenne

@RanceZero
Copy link
Author

Tested both @CheyenneWills and @FichteFoll test steps here on build 2054. Issue was not reproduced. Seems fixed @dpjohnst .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commit dialog An issue related to the commit dialog enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants