Looks good, modulo perhaps these fixes on top. - It would be better to say "dropped" to stress that we inspect and actively discard, instead of saying "ignored". - There is no need to use "changeset", the term glossary even discourages, to make sense of the sentence. - There was still a stray keep_if_made_empty. - s/Add keep/add keep/ on the title as well. No need to resend, unless you have fixes other than the following does. diff --git a/Documentation/git-cherry-pick.txt b/Documentation/git-cherry-pick.txt index 0c004e9..3d25a20 100644 --- a/Documentation/git-cherry-pick.txt +++ b/Documentation/git-cherry-pick.txt @@ -113,12 +113,12 @@ effect to your index in a row. even without this option. Note also, that use of this option only keeps commits that were initially empty (i.e. the commit recorded the same tree as its parent). Commits which are made empty due to a - previous commit are ignored. To force the inclusion of those commits + previous commit are dropped. To force the inclusion of those commits use `--keep-redundant-commits`. --keep-redundant-commits:: If a commit being cherry picked duplicates a commit already in the - current history, it will result in an empty changeset. By default these + current history, it will become empty. By default these redundant commits are ignored. This option overrides that behavior and creates an empty commit object. Implies `--allow-empty`. diff --git a/builtin/revert.c b/builtin/revert.c index f135502..b0b9b1a 100644 --- a/builtin/revert.c +++ b/builtin/revert.c @@ -141,7 +141,7 @@ static void parse_args(int argc, const char **argv, struct replay_opts *opts) "--abort", rollback, NULL); - /* keep_if_made_empty implies allow_empty */ + /* implies allow_empty */ if (opts->keep_redundant_commits) opts->allow_empty = 1; -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html