# [PATCH] doc: remove unnecessary comma in git-rm documentation

7 messages from 2026-09-23 to 2026-10-01. Participants: parovozik, Junio C Hamano, Khan Zimov.
Thread: https://gitlist.dev/t/66380

## parovozik, 2026-09-23 21:31

Subject: [PATCH] doc: remove unnecessary comma in git-rm documentation
Message-ID: <20260923213125.1316-1-kaliugov@gmail.com>

```
Signed-off-by: parovozik <kaliugov@gmail.com>
---
 Documentation/git-rm.adoc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
index b5ead86796..67061e961f 100644
--- a/Documentation/git-rm.adoc
+++ b/Documentation/git-rm.adoc
@@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 `--cached`::
-- 
2.52.0.windows.1


```

## Junio C Hamano, 2026-09-23 22:00

Subject: Re: [PATCH] doc: remove unnecessary comma in git-rm documentation
Message-ID: <xmqqo6dn6329.fsf@gitster.g>
In-Reply-To: <20260923213125.1316-1-kaliugov@gmail.com>

```
parovozik <kaliugov@gmail.com> writes:

> Signed-off-by: parovozik <kaliugov@gmail.com>

Documentation/SubmittingPatches::[[real-name]]

> ---
>  Documentation/git-rm.adoc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

$ git grep -C1 'separate command-line' Documentation/

shows that ...

> diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
> index b5ead86796..67061e961f 100644
> --- a/Documentation/git-rm.adoc
> +++ b/Documentation/git-rm.adoc
> @@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
>  
>  `--`::
>  	This option can be used to separate command-line options from
> -	the list of files, (useful when filenames might be mistaken
> +	the list of files (useful when filenames might be mistaken
>  	for command-line options).

... an identical text appears once more elsewhere.  It would be a
good idea to keep them in sync.

>  
>  `--cached`::

```

## parovozik, 2026-09-24 07:40

Subject: [PATCH v2] doc: remove unnecessary commas in git-add and git-rm documentation
Message-ID: <20260924074041.1428-1-kaliugov@gmail.com>
In-Reply-To: <xmqqo6dn6329.fsf@gitster.g>

```
Signed-off-by: parovozik <kaliugov@gmail.com>
---
 Documentation/git-add.adoc | 2 +-
 Documentation/git-rm.adoc  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/git-add.adoc b/Documentation/git-add.adoc
index 16b06e38e1..3fc2513a2d 100644
--- a/Documentation/git-add.adoc
+++ b/Documentation/git-add.adoc
@@ -223,7 +223,7 @@ for `git add --no-all <pathspec>...`, i.e. ignored removed files.
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 
diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
index b5ead86796..67061e961f 100644
--- a/Documentation/git-rm.adoc
+++ b/Documentation/git-rm.adoc
@@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 `--cached`::
-- 
2.52.0.windows.1


```

## Junio C Hamano, 2026-09-24 16:20

Subject: Re: [PATCH v2] doc: remove unnecessary commas in git-add and git-rm documentation
Message-ID: <xmqqh5je4o45.fsf@gitster.g>
In-Reply-To: <20260924074041.1428-1-kaliugov@gmail.com>

```
parovozik <kaliugov@gmail.com> writes:

Here at the top of the message before your sign-off can be left
blank for a trivial change like this patch whose motivation is so
obviously good and whose correctness is so clear.

> Signed-off-by: parovozik <kaliugov@gmail.com>

Documentation/SubmittingPatches::[[real-name]]; we prefer to see a
name that is not a "handle".

> ---

For a single-patch topic, here under the three-dash line is where
you describe what improvements were made relative to previous
iteration(s) of your work.

>  Documentation/git-add.adoc | 2 +-
>  Documentation/git-rm.adoc  | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/git-add.adoc b/Documentation/git-add.adoc
> index 16b06e38e1..3fc2513a2d 100644
> --- a/Documentation/git-add.adoc
> +++ b/Documentation/git-add.adoc
> @@ -223,7 +223,7 @@ for `git add --no-all <pathspec>...`, i.e. ignored removed files.
>  
>  `--`::
>  	This option can be used to separate command-line options from
> -	the list of files, (useful when filenames might be mistaken
> +	the list of files (useful when filenames might be mistaken
>  	for command-line options).
>  
>  
> diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
> index b5ead86796..67061e961f 100644
> --- a/Documentation/git-rm.adoc
> +++ b/Documentation/git-rm.adoc
> @@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
>  
>  `--`::
>  	This option can be used to separate command-line options from
> -	the list of files, (useful when filenames might be mistaken
> +	the list of files (useful when filenames might be mistaken
>  	for command-line options).
>  
>  `--cached`::

```

## Khan Zimov, 2026-09-29 21:02

Subject: [PATCH v3] doc: remove unnecessary commas in git-add and git-rm documentation
Message-ID: <20260929210249.1024-1-kaliugov@gmail.com>
In-Reply-To: <xmqqh5je4o45.fsf@gitster.g>

```
Signed-off-by: Khan Zimov <kaliugov@gmail.com>
---
 Documentation/git-add.adoc | 2 +-
 Documentation/git-rm.adoc  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/git-add.adoc b/Documentation/git-add.adoc
index 16b06e38e1..3fc2513a2d 100644
--- a/Documentation/git-add.adoc
+++ b/Documentation/git-add.adoc
@@ -223,7 +223,7 @@ for `git add --no-all <pathspec>...`, i.e. ignored removed files.
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 
diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
index b5ead86796..67061e961f 100644
--- a/Documentation/git-rm.adoc
+++ b/Documentation/git-rm.adoc
@@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 `--cached`::
-- 
2.52.0.windows.1


```

## Khan Zimov, 2026-09-29 21:05

Subject: [PATCH v4] doc: remove unnecessary commas in git-add and git-rm documentation
Message-ID: <20260929210618.147-1-kaliugov@gmail.com>
In-Reply-To: <xmqqh5je4o45.fsf@gitster.g>

```
Signed-off-by: Khan Zimov <kaliugov@gmail.com>
---
Changes since v3:
 - Add changes description.

Changes since v2:
 - Use real name in Signed-off-by (sorry, handle before).

Changes since v1:
 - Also fix the same text in git-add.adoc, so that both files
   stay in sync.

 Documentation/git-add.adoc | 2 +-
 Documentation/git-rm.adoc  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/git-add.adoc b/Documentation/git-add.adoc
index 16b06e38e1..3fc2513a2d 100644
--- a/Documentation/git-add.adoc
+++ b/Documentation/git-add.adoc
@@ -223,7 +223,7 @@ for `git add --no-all <pathspec>...`, i.e. ignored removed files.
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 
diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
index b5ead86796..67061e961f 100644
--- a/Documentation/git-rm.adoc
+++ b/Documentation/git-rm.adoc
@@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
 
 `--`::
 	This option can be used to separate command-line options from
-	the list of files, (useful when filenames might be mistaken
+	the list of files (useful when filenames might be mistaken
 	for command-line options).
 
 `--cached`::
-- 
2.52.0.windows.1


```

## Junio C Hamano, 2026-10-01 15:31

Subject: Re: [PATCH v4] doc: remove unnecessary commas in git-add and git-rm documentation
Message-ID: <xmqqwls177yo.fsf@gitster.g>
In-Reply-To: <20260929210618.147-1-kaliugov@gmail.com>

```
Khan Zimov <kaliugov@gmail.com> writes:

> Signed-off-by: Khan Zimov <kaliugov@gmail.com>
> ---

Thanks.  Both changes look good.  Let me mark the topic for 'next'.

> diff --git a/Documentation/git-add.adoc b/Documentation/git-add.adoc
> index 16b06e38e1..3fc2513a2d 100644
> --- a/Documentation/git-add.adoc
> +++ b/Documentation/git-add.adoc
> @@ -223,7 +223,7 @@ for `git add --no-all <pathspec>...`, i.e. ignored removed files.
>  
>  `--`::
>  	This option can be used to separate command-line options from
> -	the list of files, (useful when filenames might be mistaken
> +	the list of files (useful when filenames might be mistaken
>  	for command-line options).
>  
>  
> diff --git a/Documentation/git-rm.adoc b/Documentation/git-rm.adoc
> index b5ead86796..67061e961f 100644
> --- a/Documentation/git-rm.adoc
> +++ b/Documentation/git-rm.adoc
> @@ -61,7 +61,7 @@ For more details, see the _<pathspec>_ entry in linkgit:gitglossary[7].
>  
>  `--`::
>  	This option can be used to separate command-line options from
> -	the list of files, (useful when filenames might be mistaken
> +	the list of files (useful when filenames might be mistaken
>  	for command-line options).
>  
>  `--cached`::

```
