From: Kristoffer Haugsbakk <code@xxxxxxxxxxxxxxx> Clearly state when which of the regular and negated form of the option take effect.[1] [1]: https://lore.kernel.org/git/xmqqcyct1mtq.fsf@gitster.g/ Signed-off-by: Kristoffer Haugsbakk <code@xxxxxxxxxxxxxxx> --- Notes (series): v2: • New • The `--` are to prevent the new paragraph from becoming a continuation of the last bullet point • Junio pointed out that I didn’t go over the part about checking which variant (regular or negation) are the default. See the link: https://lore.kernel.org/git/xmqqcyct1mtq.fsf@gitster.g/ I decided to do this on this series. Link: https://lore.kernel.org/git/xmqqfrgzhzbp.fsf@gitster.g/ • The last part of the paragraph is based on the commit message of c4e2aa7d45f (notes.c: introduce "--[no-]stripspace" option, 2023-05-27). See the review on the patch: https://lore.kernel.org/git/xmqq4jp326oj.fsf@gitster.g/ There was concern about the order of options: > ... One more thing need to note is "the order of > the options matter", [...] This sounds more like a design/implementation mistake that we may want to fix. This new part is supposed to address that. Documentation/git-notes.adoc | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/Documentation/git-notes.adoc b/Documentation/git-notes.adoc index 383e8bca685..bad83116329 100644 --- a/Documentation/git-notes.adoc +++ b/Documentation/git-notes.adoc @@ -179,10 +179,18 @@ OPTIONS Clean up whitespace. Specifically (see linkgit:git-stripspace[1]): + +-- - remove trailing whitespace from all lines - collapse multiple consecutive empty lines into one empty line - remove empty lines from the beginning and end of the input - add a missing `\n` to the last line if necessary. +-- ++ +`--stripspace` is the default except for +`-C`/`--reuse-message`. However, keep in mind that this depends on the +order of similar options. For example, for `-C <object> -m<message>`, +`--stripspace` will be used because the default for `-m` overrides the +previous `-C`. `--ref <ref>`:: Manipulate the notes tree in _<ref>_. This overrides -- 2.49.0.780.g892193c3f50
