Hi Junio, On Fri, 26 Feb 2021 at 02:03, Junio C Hamano <gitster@xxxxxxxxx> wrote: > [...] > > + if (argc) > > + die(_("cannot combine reword option of --fixup with paths")); > > It would be easier if the user is told "foo" in the message when > > $ git commit --fixup=reword:HEAD~ -- foo > > (from your tests) is attempted, no? > Okay, will print the passed argument in the message. > > I am not sure this comment is all that helpful to the translaters. > If they are not allowed to translate <amend|reword> part, telling > them what that part means does not help them very much. > > Leave "[(amend|reword):]" as-is, and only translate <commit>. > > would be more direct without distracting them with useless piece of > information, no? Yes, I agree and will correct it.