Re: [PATCH] rebase -i: improve error message when picking merge

[phillip.wood123@gmail.com]

Hi Rubén

Thanks for trying this out.

On 04/04/2024 20:44, Rubén Justo wrote:
> On Mon, Feb 26, 2024 at 10:58:07AM +0000, Phillip Wood via GitGitGadget wrote:
> The change makes sense, but this is confusing to me:
> 
> With this ...
> 
> 	$ GIT_EDITOR='echo pick 17381ab62a >' ./git rebase -i HEAD
> 	error: 'pick' does not accept merge commits, please use 'merge -C'
> 	error: invalid line 1: pick 17381ab62a
> 	You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.
> 	Or you can abort the rebase with 'git rebase --abort'.
> 
> ... I find these repeated messages confusing:
> 
> 	$ GIT_EDITOR=: ./git rebase --edit-todo
> 	error: 'pick' does not accept merge commits, please use 'merge -C'
> 	error: invalid line 1: pick 17381ab62a

The two lines above are printed when "rebase --edit-todo" loads the todo 
list for the user to edit. With a real editor rather than ":" we would 
then print

     hint: Waiting for your editor to close the file ...

Then hopefully the user would fix the errors. If not then when the 
editor had finished we'd print the remaining errors  as below.

> 	error: 'pick' does not accept merge commits, please use 'merge -C'
> 	error: invalid line 1: pick 17381ab62a
> 	You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.
> 	Or you can abort the rebase with 'git rebase --abort'.

I think that printing the error messages when the todo list is parsed 
before it is given to the user to edit is helpful as it reminds the user 
what the problem is. Your test looks confusing because it doesn't really 
simulate the user editing the todo list.

Best Wishes

Phillip