Subject: Re: [PATCH] sequencer: allow disabling conflict advice

From: Junio C Hamano <[email-masked]>
Date: Sat, 09 Mar 2024 15:19:44 -0800

Junio C Hamano <[email-masked]> writes:

> Philippe Blain <[email-masked]> writes:
>
>> Thinking about this more and looking at the code, using 'advice_enabled' in the condition
>> instead of using 'advise_if_enabled' for each message has a side effect:
>> the text "hint: Disable this message with "git config advice.sequencerConflict false"
>> will not appear, which I find less user friendly...
>
> Good eyes.
>
> I agree that you have to do that part yourself at the end of that
> "if () { ... }" block using turn_off_instructions[] string.

Forgot to add "... which is an unnecessary chore" at the end.

Thanks.