Re: [PATCH] doc: replace 3 dash with correct 2 dash in git-config(1)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Aug 23, 2024 at 01:20:41AM +0800, Celeste Liu wrote:
> Commit 4e51389000 (builtin/config: introduce "get" subcommand, 2024-05-06)
> introduced this typo.

We typically do not word the commit message as a continuation of the
subject. The subject often becomes invisible in many contexts, so it is
helpful if it can be read standalone while providing all of the context
necessary. So mentioning what is wrong would be welcome here.

> Fixes: 4e51389000 (builtin/config: introduce "get" subcommand, 2024-05-06)

We do not use Fixes tags in the Git project.

> Signed-off-by: Celeste Liu <CoelacanthusHex@xxxxxxxxx>
> ---
>  Documentation/git-config.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt
> index 65c645d461..79360328aa 100644
> --- a/Documentation/git-config.txt
> +++ b/Documentation/git-config.txt
> @@ -130,7 +130,7 @@ OPTIONS
>  --all::
>  	With `get`, return all values for a multi-valued key.
>  
> ----regexp::
> +--regexp::
>  	With `get`, interpret the name as a regular expression. Regular
>  	expression matching is currently case-sensitive and done against a
>  	canonicalized version of the key in which section and variable names

This looks obviously correct to me.

There's one more thing I noticed: the synopsis for `git config get` says
`--regrexp=<regexp>`, which is clearly wrong. As you're already at it,
would you mind including a second patch that corrects this?

Thanks!

Patrick




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux