Thread (1 message) 1 message, 1 author, 2024-02-22

Re: [PATCH v2] rebase: make warning less passive aggressive

From: Junio C Hamano <hidden>
Date: 2024-02-22 17:00:13

"Kristoffer Haugsbakk" [off-list ref] writes:
I’ve interpreted this error as: I’m not quite sure but it looks like you
are not in the middle of a rebase? Like it can’t be completely
sure. Maybe from a more “primitive” time, implementation-wise?[1] :)
It certainly is a good point.  At least one other message in the
same command is equally less assertive, e.g.

	die(_("It looks like 'git am' is in progress. Cannot rebase."));

It may have been OK to be uncertain in the earlier times back when
we did not much know what we were doing, but by now we should be
more assertive, so that the end-users can more easily complain,
saying "The command said I was not rebasing, but I was!  Here is
what I did", which would hopefully be a valuable feedback for
improving the logic to diagnose what state we are in.

So, making it clear that it is a statement of a fact, or at least we
are telling the users what we _think_ the state is assertively, is a
good idea either way.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help