# [PATCH] Documentation/git-interpret-trailers: explain possible values 3 messages from 2018-07-20 to 2018-07-20. Participants: Stefan Beller, Junio C Hamano, Christian Couder. Thread: https://gitlist.dev/t/48930 ## Stefan Beller, 2018-07-20 21:53 Subject: [PATCH] Documentation/git-interpret-trailers: explain possible values Message-ID: <20180720215349.179542-1-sbeller@google.com> URL: https://gitlist.dev/e/20180720215349.179542-1-sbeller%40google.com ``` 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 ``` ## Junio C Hamano, 2018-07-20 22:23 Subject: Re: [PATCH] Documentation/git-interpret-trailers: explain possible values Message-ID: URL: https://gitlist.dev/e/xmqq1sbxa6mv.fsf%40gitster-ct.c.googlers.com In-Reply-To: <20180720215349.179542-1-sbeller@google.com> ``` 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. ``` ## Christian Couder, 2018-07-20 23:44 Subject: Re: [PATCH] Documentation/git-interpret-trailers: explain possible values Message-ID: URL: https://gitlist.dev/e/CAP8UFD3vyphLP0P1Y2oDCCcz--qXz2G1%3Dr9X1iVnq%3DbwuS9%2B3w%40mail.gmail.com In-Reply-To: ``` 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. ```