Re: [PATCH v8] ls-files: introduce "--format" option
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Jul 21, 2022, 17:22 UTC
- Message-ID
- <CAPig+cTAnVjk9gepW33AW9SSpCTJ7FpNSaB_7M-wV+tTe6D04A@mail.gmail.com>
- In-Reply-To
- <220721.86ilnqtq8f.gmgdl@evledraar.gmail.com>
On Thu, Jul 21, 2022 at 12:07 PM Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:
Show 13 quoted lines
> On Wed, Jul 20 2022, Junio C Hamano wrote:
> > "ZheNing Hu via GitGitGadget" <gitgitgadget@gmail.com> writes:
> > It's been quite many iterations, so I'll just comment on the range-diff.
> >
> >> -+ usage_msg_opt("--format cannot used with -s, -o, -k, -t"
> >> ++ usage_msg_opt("--format cannot used with -s, -o, -k, -t, "
> >> + "--resolve-undo, --deduplicate, --eol",
> >> + ls_files_usage, builtin_ls_files_options);
> >
> > Looks good.
>
> Although a nit I didn't spot before: missing _() & this should be marked
> for translation, surely...Probably also want: s/cannot used/cannot be used/