Re: [PATCH] help: add space after autocorrect prompt

From: Junio C Hamano <gitster@pobox.com>
Date: Dec 16, 2021, 21:32 UTC
Message-ID: <xmqq35ms9qxg.fsf@gitster.g>
In-Reply-To: <20211216062709.b4tcagi6qcqhasrh@gmail.com>

Johannes Altmanninger <aclopte@gmail.com> writes:

Show 12 quoted lines

>>  - It does not leave a SP before where the end-user input goes;
>>  - It append (y/N) _after_ question mark;
>
> I actually prefer this one;
>
> 	Run 'status' instead? [y/N] 
>
> sounds better than
>
> 	Run 'status' instead [y/N]? 
>
> but that's not a strong opinion.

It is fine to have your own preference, but we are discussing to make the three, two of which use "[choices]? " while only one uses "? (choices)", consistent.

So I'd prefer to see us turn the odd man out to do "[choices]? " first. Adjusting to your personal preference can be proposed after the dust settles for the consistency fix.

Thanks.