Re: [PATCH v2 0/9] rebase -i: add config to abbreviate command names
On 05/12/17 05:21 PM, Junio C Hamano wrote: > Liam Beguinwrites: > >> This series will add the 'rebase.abbreviateCommands' configuration >> option to allow `git rebase -i` to default to the single-letter command >> names when generating the todo list. >> >> Using single-letter command names can present two benefits. First, it >> makes it easier to change the action since you only need to replace a >> single character (i.e.: in vim "r" instead of >> "ciw"). Second, using this with a large enough value of >> 'core.abbrev' enables the lines of the todo list to remain aligned >> making the files easier to read. >> >> Changes in V2: >> - Refactor and rename 'transform_todo_ids' >> - Replace SHA-1 by OID in rebase--helper.c >> - Update todo list related functions to take a generic 'flags' parameter >> - Rename 'add_exec_commands' function to 'sequencer_add_exec_commands' >> - Rename 'add-exec' option to 'add-exec-commands' >> - Use 'strbur_read_file' instead of rewriting it >> - Make 'command_to_char' return 'comment_char_line' if no single-letter >> command name is defined >> - Combine both tests into a single test case >> - Update commit messages >> >> Changes in V2: >> - Rename 'transform_todo_insn' to 'transform_todos' >> - Fix flag name TODO_LIST_SHORTE{D,N}_IDS > > I've replaced this series and pushed out the result. Great! Thanks again, > > Thanks. > Liam
Re: [PATCH v2 0/9] rebase -i: add config to abbreviate command names
Liam Beguinwrites: > This series will add the 'rebase.abbreviateCommands' configuration > option to allow `git rebase -i` to default to the single-letter command > names when generating the todo list. > > Using single-letter command names can present two benefits. First, it > makes it easier to change the action since you only need to replace a > single character (i.e.: in vim "r" instead of > "ciw"). Second, using this with a large enough value of > 'core.abbrev' enables the lines of the todo list to remain aligned > making the files easier to read. > > Changes in V2: > - Refactor and rename 'transform_todo_ids' > - Replace SHA-1 by OID in rebase--helper.c > - Update todo list related functions to take a generic 'flags' parameter > - Rename 'add_exec_commands' function to 'sequencer_add_exec_commands' > - Rename 'add-exec' option to 'add-exec-commands' > - Use 'strbur_read_file' instead of rewriting it > - Make 'command_to_char' return 'comment_char_line' if no single-letter > command name is defined > - Combine both tests into a single test case > - Update commit messages > > Changes in V2: > - Rename 'transform_todo_insn' to 'transform_todos' > - Fix flag name TODO_LIST_SHORTE{D,N}_IDS I've replaced this series and pushed out the result. Thanks.
Re: [PATCH v2 0/9] rebase -i: add config to abbreviate command names
Hi Liam, On Sun, 3 Dec 2017, Liam Beguin wrote: > This series will add the 'rebase.abbreviateCommands' configuration > option to allow `git rebase -i` to default to the single-letter command > names when generating the todo list. > > Using single-letter command names can present two benefits. First, it > makes it easier to change the action since you only need to replace a > single character (i.e.: in vim "r" instead of > "ciw"). Second, using this with a large enough value of > 'core.abbrev' enables the lines of the todo list to remain aligned > making the files easier to read. > > Changes in V2: > - Refactor and rename 'transform_todo_ids' > - Replace SHA-1 by OID in rebase--helper.c > - Update todo list related functions to take a generic 'flags' parameter > - Rename 'add_exec_commands' function to 'sequencer_add_exec_commands' > - Rename 'add-exec' option to 'add-exec-commands' > - Use 'strbur_read_file' instead of rewriting it > - Make 'command_to_char' return 'comment_char_line' if no single-letter > command name is defined > - Combine both tests into a single test case > - Update commit messages Looks very nice already! I offered a couple of comments/suggestions, but nothing major. Thank you, Johannes