Re: [PATCH 1/2] docs: Fix bad whitespace in Documentation/config/pgp.txt

From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: Greg Hurrell <greg@hurrell.net>
Cc: Git Mailing List <git@vger.kernel.org>
Subject: Re: [PATCH 1/2] docs: Fix bad whitespace in Documentation/config/pgp.txt
Date: Tue, 25 Jan 2022 18:09:57 +0100	[thread overview]
Message-ID: <220125.861r0vhh8r.gmgdl@evledraar.gmail.com> (raw)
In-Reply-To: <20220125123716.66991-2-greg@hurrell.net>

On Tue, Jan 25 2022, Greg Hurrell wrote:

nit @ subject: I'd suggest:

	Documentation/config/pgp.txt: replace stray <TAB> character with <SPC>

Or something, i.e. the "docs" can just be replaced by the filename, so
we don't need to put it at the end.

As general style, we tend to not have the <msg> part of "<subsystem>:
<msg>" start with a capital letter, i.e. "fix" not "Fix".

> Signed-off-by: Greg Hurrell <greg@hurrell.net>
> ---
>  Documentation/config/gpg.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/config/gpg.txt b/Documentation/config/gpg.txt
> index 0cb189a077..abfabd6d62 100644
> --- a/Documentation/config/gpg.txt
> +++ b/Documentation/config/gpg.txt
> @@ -37,7 +37,7 @@ gpg.minTrustLevel::
>  gpg.ssh.defaultKeyCommand::
>  	This command that will be run when user.signingkey is not set and a ssh
>  	signature is requested. On successful exit a valid ssh public key is
> -	expected in the	first line of its output. To automatically use the first
> +	expected in the first line of its output. To automatically use the first

Since I had to look it's a s/\t/ / of a tab between "the" and "first" in
the pre-image>