threads / patch / 61229

patchconfig: fix some small capitalization issues, as spotted

Subject: [PATCH] config: fix some small capitalization issues, as spotted

## tl;dr

3 messages between Mar 30, 2024 and Mar 31, 2024. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Dragan Simic· Mar 30, 2024, 04:26 UTC · lore

Fix some small capitalization issues, as spotted while going through the documentation. In general, a semicolon doesn't start a new sentence, and "this" has no meaning of a proper noun in this context.

Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---

Notes:
     * Fix some small capitalization issues in the documentation.

 Documentation/config/core.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to Documentation/config/core.txt +1 −1

diff --git a/Documentation/config/core.txt b/Documentation/config/core.txt
index 2d4bbdb25fa3..31fe417065c6 100644
--- a/Documentation/config/core.txt
+++ b/Documentation/config/core.txt
@@ -688,7 +688,7 @@ core.createObject::
 	will not overwrite existing objects.
 +
 On some file system/operating system combinations, this is unreliable.
-Set this config setting to 'rename' there; However, This will remove the
+Set this config setting to 'rename' there;  however, this will remove the
 check that makes sure that existing object files will not get overwritten.
 
 core.notesRef::

Junio C Hamano· Mar 30, 2024, 20:34 UTC · re: Dragan Simic · lore

Re: [PATCH] config: fix some small capitalization issues, as spotted

Dragan Simic <dsimic@manjaro.org> writes:

>  On some file system/operating system combinations, this is unreliable.
> -Set this config setting to 'rename' there; However, This will remove the
> +Set this config setting to 'rename' there;  however, this will remove the
>  check that makes sure that existing object files will not get overwritten.

Looking good, except that we do not want double-space after a semicolon.

Dragan Simic· Mar 31, 2024, 20:21 UTC · re: Junio C Hamano · lore

Re: [PATCH] config: fix some small capitalization issues, as spotted

Hello Junio,

On 2024-03-30 21:34, Junio C Hamano wrote:

Show 13 quoted lines

> Dragan Simic <dsimic@manjaro.org> writes:
> 
>>  On some file system/operating system combinations, this is 
>> unreliable.
>> -Set this config setting to 'rename' there; However, This will remove 
>> the
>> +Set this config setting to 'rename' there;  however, this will remove 
>> the
>>  check that makes sure that existing object files will not get 
>> overwritten.
> 
> Looking good, except that we do not want double-space after a
> semicolon.

Thanks, I'll send a v2 that fixes this issue.