From: Patrick Steinhardt <ps@pks.im>
Date: Tue, 27 Jan 2026 06:58:41 GMT
Subject: Re: [PATCH v5 2/2] repo: add new flag --keys to git-repo-info
Message-ID: <aXhiIQXBvMhzkFy9@pks.im>
In-Reply-To: <20260123164900.35092-3-lucasseikioshiro@gmail.com>

On Fri, Jan 23, 2026 at 01:34:54PM -0300, Lucas Seiki Oshiro wrote:
> diff --git a/Documentation/git-repo.adoc b/Documentation/git-repo.adoc
> index 693e1bbced..f474274009 100644
> --- a/Documentation/git-repo.adoc
> +++ b/Documentation/git-repo.adoc
> @@ -45,6 +46,16 @@ supported:
>  +
>  `-z` is an alias for `--format=nul`.
>  
> +`info --keys [--format=(lines|nul) | -z]`::
> +	List all the available keys, one per line. The output format can be chosen
> +	through the flag `--format`. The following formats are supported:
> ++
> +`lines`:::
> +	output the keys one per line. This is the default.
> +
> +`nul`:::
> +	similar to `lines`, but using a _NUL_ character after each value.

Shouldn' these sentences start with an upper-case character? I see that
we don't do it either for the existing docs, but it reads a bit weird to
me.

Patrick