Re: [PATCH v9 4/4] cat-file: add --batch-command mode

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 18, 2022, 17:23 UTC
Message-ID: <xmqqtucw2io4.fsf@gitster.g>
In-Reply-To: <095278d4-217b-7d97-5364-9cd491b34ed0@gmail.com>

Phillip Wood <phillip.wood123@gmail.com> writes:

Show 8 quoted lines

> This is looking good. I think the only thing that is missing (and
> which I should have realized earlier) is that there are no tests for
> valid or invalid format arguments to --batch-command. I haven't
> checked but there must be some other tests in the t1006 that we can
> piggy back on to add that. I've left some stylistic comments below but
> I don't fell strongly about them apart from the README comment so
> please don't feel obliged to act on them, it's looking pretty good as
> is.

Yes, I agree that this is mostly good. I also agree with all the points you raised in your review, including the stylistic ones.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.