git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] doc: imply that interactive.singleKey is disabled by default

From
DSDragan Simic <dsimic@manjaro.org>
Date
May 31, 2024, 22:21 UTC
Message-ID
<e5eb72eeb2f8e2de9400ed9373b72236@manjaro.org>
In-Reply-To
<xmqq5xutgajw.fsf@gitster.g>
Hello Junio,
On 2024-05-31 19:23, Junio C Hamano wrote:
Show 46 quoted lines
> Dragan Simic <dsimic@manjaro.org> writes:
> 
>> On 2024-05-22 23:34, Dragan Simic wrote:
>>> On 2024-05-22 23:31, Junio C Hamano wrote:
>>>> Dragan Simic <dsimic@manjaro.org> writes:
>>>> 
>>>>> 1:  3141fe9f7328 ! 1:  7da73f15a018 doc: note that
>>>>> interactive.singleKey is disabled by default
>>>>>     @@ Metadata
>>>>>      Author: Dragan Simic <dsimic@manjaro.org>
>>>>>       ## Commit message ##
>>>>>     -    doc: note that interactive.singleKey is disabled by 
>>>>> default
>>>>>     +    doc: imply that interactive.singleKey is disabled by 
>>>>> default
>>>>>          Make it clear that the interactive.singleKey
>>>>> configuration option is disabled
>>>> Heh, "note that" would probably be better, as we are going to say
>>>> "Make it clear" anyway, no?  That is stronger than just to imply.
>>>> Keeping the original version of the log message probably was
>>>> sufficient.
>>> Yeah, but I felt like it needed some adjustments, because the patch
>>> itself was actually changed.  I hope I won't have to send the v3
>>> because of that? :)
>>> 
>>>>> +	When set to true, allow the user to provide one-letter input
>>>>> +	with a single key (i.e., without hitting the Enter key) in
>>>>> +	interactive commands.  This is currently used by the `--patch`
>>>>> +	mode of linkgit:git-add[1], linkgit:git-checkout[1],
>>>>>  	linkgit:git-restore[1], linkgit:git-commit[1],
>>>>>  	linkgit:git-reset[1], and linkgit:git-stash[1].
>>>>> 
>>>> The resulting text reads well.
>>>> Nicely done.
>>> Thanks.
>> 
>> Just checking, is there something left to be addressed for this patch,
>> before it can be considered to be pulled into the next branch?
> 
> Thanks for pinging, as these small things were on the back burner
> while preparing for updates to maintenance tracks.
> 
> Apparently v2 cannot be pulled into the next branch, and I forgot if
> I saw v3 already.  In general, unless I explicitly say there is no
> need to resend (sometimes with conditions), I'd expect an updated
> iteration sent to the list.

I see, but I'm not really sure is there need for the v3? Maybe the patch description could be tweaked a bit further, but I wasn't under impression that you asked for that to be done? Am I wrong there?

Previous: Junio C HamanoNext: Dragan Simic
Message 6 of 10 in “doc: imply that interactive.singleKey is disabled by default”
  1. doc: imply that interactive.singleKey is disabled by defaultDragan Simic, May 22, 2024
  2. Junio C HamanoMay 22, 2024
  3. Dragan SimicMay 22, 2024
  4. Dragan SimicMay 31, 2024
  5. Junio C HamanoMay 31, 2024
  6. Dragan SimicMay 31, 2024
  7. Dragan SimicJun 4, 2024
  8. Junio C HamanoJun 7, 2024
  9. Dragan SimicJun 8, 2024
  10. doc: interactive.singleKey is disabled by defaultJunio C Hamano, Jun 7, 2024

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.