threads / patch / 61533

v2doc: imply that interactive.singleKey is disabled by default

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

## tl;dr

10 messages between May 22, 2024 and Jun 8, 2024. Diffs are folded; open one to read it.

replies: 9people: 2as markdown or json

Dragan Simic· May 22, 2024, 21:24 UTC · lore

Make it clear that the interactive.singleKey configuration option is disabled by default, using rather subtle wording that avoids an emphasis on the actual default value. This should eliminate any associated doubts.

While there, touch up the remaining wording of the description a bit.
Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---
Notes:
    Changes in v2:
      - As agreed upon with Junio, [1] switched to more subtle wording that
        does not explicitly mention the default value.
    
    [1] https://lore.kernel.org/git/xmqqttip379x.fsf@gitster.g/
    
    Link to v1: https://lore.kernel.org/git/3141fe9f7328a97bf3818748421f6ae929120626.1716363167.git.dsimic@manjaro.org/T/#u
Range-diff against v1:
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
    -    by default.  This should eliminate any associated doubts.
    +    by default, using rather subtle wording that avoids an emphasis on the actual
    +    default value.  This should eliminate any associated doubts.
    +
    +    While there, touch up the remaining wording of the description a bit.
     
         Signed-off-by: Dragan Simic <dsimic@manjaro.org>
     
    +
    + ## Notes ##
    +    Changes in v2:
    +      - As agreed upon with Junio, [1] switched to more subtle wording that
    +        does not explicitly mention the default value.
    +
    +    [1] https://lore.kernel.org/git/xmqqttip379x.fsf@gitster.g/
    +
    +    Link to v1: https://lore.kernel.org/git/3141fe9f7328a97bf3818748421f6ae929120626.1716363167.git.dsimic@manjaro.org/T/#u
    +
      ## Documentation/config/interactive.txt ##
    -@@ Documentation/config/interactive.txt: interactive.singleKey::
    - 	linkgit:git-add[1], linkgit:git-checkout[1],
    +@@
    + interactive.singleKey::
    +-	In interactive commands, allow the user to provide one-letter
    +-	input with a single key (i.e., without hitting enter).
    +-	Currently this is used by the `--patch` mode of
    +-	linkgit:git-add[1], linkgit:git-checkout[1],
    ++	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].
    -+	This is disabled by default.
      
    - interactive.diffFilter::
    - 	When an interactive command (such as `git add --patch`) shows
 Documentation/config/interactive.txt | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
Show changes to Documentation/config/interactive.txt +4 −4
diff --git a/Documentation/config/interactive.txt b/Documentation/config/interactive.txt
index 5cc26555f19a..8b876cb4eb8e 100644
--- a/Documentation/config/interactive.txt
+++ b/Documentation/config/interactive.txt
@@ -1,8 +1,8 @@
 interactive.singleKey::
-	In interactive commands, allow the user to provide one-letter
-	input with a single key (i.e., without hitting enter).
-	Currently this is used by the `--patch` mode of
-	linkgit:git-add[1], linkgit:git-checkout[1],
+	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].
Junio C Hamano· May 22, 2024, 21:31 UTC · re: Dragan Simic · lore

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

Dragan Simic <dsimic@manjaro.org> writes:
Show 9 quoted lines
> 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.

Show 7 quoted lines
> +	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.

Dragan Simic· May 22, 2024, 21:34 UTC · re: Junio C Hamano · lore

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

On 2024-05-22 23:31, Junio C Hamano wrote:
Show 19 quoted lines
> 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? :)

Show 10 quoted lines
>> +	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.
Dragan Simic· May 31, 2024, 00:49 UTC · re: Dragan Simic · lore

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

On 2024-05-22 23:34, Dragan Simic wrote:
Show 37 quoted lines
> 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?

Junio C Hamano· May 31, 2024, 17:23 UTC · re: Dragan Simic · lore

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

Dragan Simic <dsimic@manjaro.org> writes:
Show 34 quoted lines
> 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.

Thanks.
Dragan Simic· May 31, 2024, 22:21 UTC · re: Junio C Hamano · lore

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

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?

Dragan Simic· Jun 4, 2024, 20:11 UTC · re: Dragan Simic · lore

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

On 2024-06-01 00:21, Dragan Simic wrote:
Show 17 quoted lines
> On 2024-05-31 19:23, Junio C Hamano wrote:
>> Dragan Simic <dsimic@manjaro.org> writes:
>>> 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?
Any chances, please, to have a look at this?
Junio C Hamano· Jun 7, 2024, 22:15 UTC · re: Dragan Simic · lore

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

Dragan Simic <dsimic@manjaro.org> writes:
Show 18 quoted lines
> On 2024-06-01 00:21, Dragan Simic wrote:
>> On 2024-05-31 19:23, Junio C Hamano wrote:
>>> Dragan Simic <dsimic@manjaro.org> writes:
>>>> 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?
>
> Any chances, please, to have a look at this?
I _have_ taken a look---as I said, "I saw v3 already".

Unless you were asking other folks, that is, but the message was addressed to me with others CC'ed, so I am not sure what the true intention was.

Dragan Simic· Jun 8, 2024, 09:59 UTC · re: Junio C Hamano · lore

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

Hello Junio,
On 2024-06-08 00:15, Junio C Hamano wrote:
Show 26 quoted lines
> Dragan Simic <dsimic@manjaro.org> writes:
> 
>> On 2024-06-01 00:21, Dragan Simic wrote:
>>> On 2024-05-31 19:23, Junio C Hamano wrote:
>>>> Dragan Simic <dsimic@manjaro.org> writes:
>>>>> 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?
>> 
>> Any chances, please, to have a look at this?
> 
> I _have_ taken a look---as I said, "I saw v3 already".
> 
> Unless you were asking other folks, that is, but the message was
> addressed to me with others CC'ed, so I am not sure what the true
> intention was.

Ah, I see now [1] what you asked for, which was a slight rewording of the patch subject and description. That's what my intention was, to check was rewording the patch v2 subject and description necessary. Sorry for the confusion, and thanks for putting the v3 [1] together.

[1] https://lore.kernel.org/git/xmqq4ja4e6d6.fsf@gitster.g/
Junio C Hamano· Jun 7, 2024, 22:27 UTC · re: Dragan Simic · lore

[PATCH v3] doc: interactive.singleKey is disabled by default

From: Dragan Simic <dsimic@manjaro.org>

Make it clear that the interactive.singleKey configuration option is disabled by default, using rather subtle wording that avoids an emphasis on the actual default value. This should eliminate any associated doubts.

While there, touch up the remaining wording of the description a bit.
Signed-off-by: Dragan Simic <dsimic@manjaro.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 * While I think it is pointless to say an optional on/off switch
   defaults to off, the structure of the paragraph is better with
   the updated text overall, so let's take this version before we
   forget.
 Documentation/config/interactive.txt | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
Show changes to Documentation/config/interactive.txt +4 −4
diff --git a/Documentation/config/interactive.txt b/Documentation/config/interactive.txt
index 5cc26555f1..8b876cb4eb 100644
--- a/Documentation/config/interactive.txt
+++ b/Documentation/config/interactive.txt
@@ -1,8 +1,8 @@
 interactive.singleKey::
-	In interactive commands, allow the user to provide one-letter
-	input with a single key (i.e., without hitting enter).
-	Currently this is used by the `--patch` mode of
-	linkgit:git-add[1], linkgit:git-checkout[1],
+	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].
 

-- 
2.45.2-445-g1b76f06508

← back to recent threads