Elijah Newren <newren@xxxxxxxxx> writes: > diff --git a/Documentation/merge-options.txt b/Documentation/merge-options.txt > index 79a00d2a4a..ed3804650b 100644 > --- a/Documentation/merge-options.txt > +++ b/Documentation/merge-options.txt > @@ -40,20 +40,26 @@ set to `no` at the beginning of them. > case of a merge conflict. > > --ff:: > --no-ff:: > --ff-only:: > + Whether to prefer resolving the merge as a fast forward (only > + updating the branch pointer to match the merged branch and not > + creating a merge commit), to never allow it (always creating a > + merge commit), or to only allow fast forward updates. The > + default is `--ff`, except when merging an annotated (and > + possibly signed) tag that is not stored in its natural place > + in the `refs/tags/` hierarchy (in which case `--no-ff` is > + assumed). > ++ > +With `--ff`, resolve the merge as a fast-forward when possible (when the > +merged branch contains the current branch in its history). When not > +possible, create a merge commit. > ++ > +With `--no-ff`, create a merge commit in all cases, even when the merge > +could instead be resolved as a fast-forward. > ++ > +With `--ff-only`, resolve the merge as a fast-forward when possible. > +When not possible, refuse to merge and exit with a non-zero status. I cannot shake the feeling that the above redundantly repeats the same thing twice. If we want to dedicate one paragraph for each of these options, we can and should make the introductory paragraph lighter by saying something like Specifies how a merge is handled when the merged-in history is already a descendant of the current history. `--ff` is the default unless merging an annotated or signed tag that is not stored in the `refs/tags/` hierarchy, in which case `--no-ff` is the default. Alternatively, we could sprinkle the actual option name in the first paragraph and drop the last three paragraphs, while fattening the description as necessary, e.g. Whether to prefer resolving the merge as a fast-forward and update the branch pointer to match the merged branch without creating an extra merge commit (`--ff`), never allow fast-forward and always creating an extra merge commit (`--no-ff`), or to only allow fast forward updates and reject when a merge commit needs to be created (`--ff-only`). The default is ... I think either approach shown above would reduce the redundancy. I do not care too deeply which one of these approaches is used myself, but the redundancy feels a bit disturbing. Thanks.
