Re: [PATCH] Documentation/git-interpret-trailers: explain possible values

2018-07-20 Thread Christian Couder
On Sat, Jul 21, 2018 at 12:23 AM, Junio C Hamano  wrote:
> Stefan Beller  writes:
>
>> Signed-off-by: Stefan Beller 
>> ---
>>
>> Maybe we rather want to refer to the options that are described further
>> down in the document?
>
> I have no strong preference either way.
>
> The patch looks reasonable to me; Christian?

Yeah, it looks reasonable to me too.

I wouldn't mind also referring to the options described elsewhere, but
it could be in another patch.


Re: [PATCH] Documentation/git-interpret-trailers: explain possible values

2018-07-20 Thread Junio C Hamano
Stefan Beller  writes:

> Signed-off-by: Stefan Beller 
> ---
>
> Maybe we rather want to refer to the options that are described further
> down in the document?

I have no strong preference either way.

The patch looks reasonable to me; Christian?

>
>  Documentation/git-interpret-trailers.txt | 9 ++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/git-interpret-trailers.txt 
> b/Documentation/git-interpret-trailers.txt
> index 9111c47a1bf..b8fafb1e8bd 100644
> --- a/Documentation/git-interpret-trailers.txt
> +++ b/Documentation/git-interpret-trailers.txt
> @@ -88,7 +88,8 @@ OPTIONS
>   Specify where all new trailers will be added.  A setting
>   provided with '--where' overrides all configuration variables
>   and applies to all '--trailer' options until the next occurrence of
> - '--where' or '--no-where'.
> + '--where' or '--no-where'. Possible values are `after`, `before`,
> + `end` or `start`.
>  
>  --if-exists ::
>  --no-if-exists::
> @@ -96,7 +97,8 @@ OPTIONS
>   least one trailer with the same  in the message.  A setting
>   provided with '--if-exists' overrides all configuration variables
>   and applies to all '--trailer' options until the next occurrence of
> - '--if-exists' or '--no-if-exists'.
> + '--if-exists' or '--no-if-exists'. Possible actions are 
> `addIfDifferent`,
> + `addIfDifferentNeighbor`, `add`, `replace` and `doNothing`.
>  
>  --if-missing ::
>  --no-if-missing::
> @@ -104,7 +106,8 @@ OPTIONS
>   trailer with the same  in the message.  A setting
>   provided with '--if-missing' overrides all configuration variables
>   and applies to all '--trailer' options until the next occurrence of
> - '--if-missing' or '--no-if-missing'.
> + '--if-missing' or '--no-if-missing'. Possible actions are `doNothing`
> + or `add`.
>  
>  --only-trailers::
>   Output only the trailers, not any other parts of the input.


[PATCH] Documentation/git-interpret-trailers: explain possible values

2018-07-20 Thread Stefan Beller
Signed-off-by: Stefan Beller 
---

Maybe we rather want to refer to the options that are described further
down in the document?

 Documentation/git-interpret-trailers.txt | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/Documentation/git-interpret-trailers.txt 
b/Documentation/git-interpret-trailers.txt
index 9111c47a1bf..b8fafb1e8bd 100644
--- a/Documentation/git-interpret-trailers.txt
+++ b/Documentation/git-interpret-trailers.txt
@@ -88,7 +88,8 @@ OPTIONS
Specify where all new trailers will be added.  A setting
provided with '--where' overrides all configuration variables
and applies to all '--trailer' options until the next occurrence of
-   '--where' or '--no-where'.
+   '--where' or '--no-where'. Possible values are `after`, `before`,
+   `end` or `start`.
 
 --if-exists ::
 --no-if-exists::
@@ -96,7 +97,8 @@ OPTIONS
least one trailer with the same  in the message.  A setting
provided with '--if-exists' overrides all configuration variables
and applies to all '--trailer' options until the next occurrence of
-   '--if-exists' or '--no-if-exists'.
+   '--if-exists' or '--no-if-exists'. Possible actions are 
`addIfDifferent`,
+   `addIfDifferentNeighbor`, `add`, `replace` and `doNothing`.
 
 --if-missing ::
 --no-if-missing::
@@ -104,7 +106,8 @@ OPTIONS
trailer with the same  in the message.  A setting
provided with '--if-missing' overrides all configuration variables
and applies to all '--trailer' options until the next occurrence of
-   '--if-missing' or '--no-if-missing'.
+   '--if-missing' or '--no-if-missing'. Possible actions are `doNothing`
+   or `add`.
 
 --only-trailers::
Output only the trailers, not any other parts of the input.
-- 
2.18.0.233.g985f88cf7e-goog