git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: git rebase interactive: usability issue

From
Avery Pennarun <apenwarr@gmail.com>
Date
Jun 26, 2008, 03:59 UTC
Message-ID
<32541b130806252059l30431fcbld6196bedb32ab397@mail.gmail.com>
In-Reply-To
<7vbq1ovpl7.fsf@gitster.siamese.dyndns.org>
On 6/25/08, Junio C Hamano <gitster@pobox.com> wrote:
>  I was relunctant about the patch not because of "edit", but because I am
>  not convinced that it will _never_ make sense to be able to amend while
>  the sequence stops with a conflict (as the patch does not give us any way
>  to override this rather heavy-handed denial to continue).

Perhaps the problem is more that people are encouraged to --amend so often that they end up doing it by accident.

What if 'edit' worked more like 'squash', in that it produced the new tree, but didn't commit it yet? Then you can reset things, commit them, or rebase --continue (which commits automatically if needed) just like wish 'squash'.

I think --continue used to not commit automatically, so I can see why edit used to commit for you, but maybe that behaviour is not needed anymore.

Right now the asymmetry of having to use --amend with 'edit' but not with 'squash' is what leads me to make mistakes sometimes.

Have fun,
Avery
Previous: Junio C HamanoNext: Dmitry Potapov
Message 4 of 7 in “git rebase interactive: usability issue”
  1. Dmitry PotapovJun 25, 2008
  2. Johannes SchindelinJun 26, 2008
  3. Junio C HamanoJun 26, 2008
  4. Avery PennarunJun 26, 2008
  5. Dmitry PotapovJun 26, 2008
  6. Theodore TsoJun 26, 2008
  7. don't allow 'commit --amend' during rebase conflict resolutionDmitry Potapov, Jun 26, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.