Re: [PATCH v2 03/11] rebase -i: clarify and fix 'fixup -c' rebase-todo help
From: Charvi Mendiratta <hidden>
Date: 2021-02-09 07:14:19
Hi Junio,
quoted
"f, fixup [-C | -c] <commit> = like \"squash\", but discard this\n" -" commit's log message. Use -C to replace with this\n" -" commit message or -c to edit the commit message\n" +" commit's log message; use -C to replace with this\n" +" commit message or -c to edit this commit message\n"The goal is good, but I am not sure if this "the commit" -> "this commit" is an effective enough way to fix the issue. Here is my attempt but I do not think it is not 10x better to be worth replacing yours X-<. use only the log message of the "fixup" commit, discarding the message from the previous commit. While "-C" uses the message as-is, "-c" lets the user further edit it.
Okay, but in this patch we are also removing period and capitalization from rebase to-do help of commands. So, maybe we can replace it like : f, fixup [-C | -c] <commit> = like \"squash\", but discard this\n" " commit's log message; use -C to use only the\n" " log message of the "fixup" commit, discarding the\n" " message from the previous commit; while -C uses \n" " the message as-is, -c allows to further edit it\n" If it is okay ? Thanks and Regards, Charvi