From: rsbecker@nexbridge.com
Date: Fri, 11 Aug 2023 17:56:14 GMT
Subject: RE: Re* [PATCH v3 2/2] doc: revert: add discussion
Message-ID: <033a01d9cc7d$2290e600$67b2b200$@nexbridge.com>
In-Reply-To: <xmqq1qg9tqhq.fsf@gitster.g>

On Friday, August 11, 2023 1:54 PM, Junio C Hamano wrote:
>Junio C Hamano <gitster@pobox.com> writes:
>
>> +
>> +		if (starts_with(msg.subject, "Reapply \"Reapply \""))
>> +			/* fifth time is too many - force reference format*/
>> +			use_reference = 1;
>
>Come to think of it, as the documentation patch in the series cited double
reapply as
>too unwieldy, we probably should stop before producing such commit.  We can
>update "Reapply \"Reapply" above to "Revert \"Reapply" and then "fifth" ->
"fourth".
>The test update below must also be adjusted, if we want to take that route.

May I suggest a potential quick solution. Perhaps we could leave this up to
users by putting in an --amend or --reword option to cause a prompt for a
comment for the reverted commit instead of trying to come up with a
one-size-fits-all solution.