# [PATCH] docs: fix typesetting of merge driver placeholders

2 messages from 2025-01-07 to 2025-01-07. Participants: Matthew Hughes, Junio C Hamano.
Thread: https://gitlist.dev/t/62764

## Matthew Hughes, 2025-01-07 21:24

Subject: [PATCH] docs: fix typesetting of merge driver placeholders
Message-ID: <20250107212421.7yyvuzw4uqxnqv7t@archP14s>
URL: https://gitlist.dev/e/20250107212421.7yyvuzw4uqxnqv7t%40archP14s

```
Following the `CodingGuidlines`, since these placeholders are literal
they should be typeset verbatim, so fix some that aren't.

Signed-off-by: Matthew Hughes <matthewhughes934@gmail.com>
---
 Documentation/gitattributes.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/gitattributes.txt b/Documentation/gitattributes.txt
index e6150595af..5d12b78549 100644
--- a/Documentation/gitattributes.txt
+++ b/Documentation/gitattributes.txt
@@ -1166,7 +1166,7 @@ internal merge and the final merge.
 The merge driver can learn the pathname in which the merged result
 will be stored via placeholder `%P`. The conflict labels to be used
 for the common ancestor, local head and other head can be passed by
-using '%S', '%X' and '%Y` respectively.
+using `%S`, `%X` and `%Y` respectively.
 
 `conflict-marker-size`
 ^^^^^^^^^^^^^^^^^^^^^^

base-commit: 14650065b76b28d3cfa9453356ac5669b19e706e
-- 
2.47.1


```

## Junio C Hamano, 2025-01-07 23:11

Subject: Re: [PATCH] docs: fix typesetting of merge driver placeholders
Message-ID: <xmqqed1eqkxm.fsf@gitster.g>
URL: https://gitlist.dev/e/xmqqed1eqkxm.fsf%40gitster.g
In-Reply-To: <20250107212421.7yyvuzw4uqxnqv7t@archP14s>

```
Matthew Hughes <matthewhughes934@gmail.com> writes:

> Following the `CodingGuidlines`, since these placeholders are literal
> they should be typeset verbatim, so fix some that aren't.
>
> Signed-off-by: Matthew Hughes <matthewhughes934@gmail.com>
> ---
>  Documentation/gitattributes.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Yup, especially the %P placeholder uses the backquote/literal
mark-up already in a few lines above, this is an obviously right
thing to do.

Will queue.  Thanks.

>
> diff --git a/Documentation/gitattributes.txt b/Documentation/gitattributes.txt
> index e6150595af..5d12b78549 100644
> --- a/Documentation/gitattributes.txt
> +++ b/Documentation/gitattributes.txt
> @@ -1166,7 +1166,7 @@ internal merge and the final merge.
>  The merge driver can learn the pathname in which the merged result
>  will be stored via placeholder `%P`. The conflict labels to be used
>  for the common ancestor, local head and other head can be passed by
> -using '%S', '%X' and '%Y` respectively.
> +using `%S`, `%X` and `%Y` respectively.
>  
>  `conflict-marker-size`
>  ^^^^^^^^^^^^^^^^^^^^^^
>
> base-commit: 14650065b76b28d3cfa9453356ac5669b19e706e

```
