Re: [PATCH v5 22/26] t7700: consolidate code into test_no_missing_in_packs()

From: Junio C Hamano <gitster@pobox.com>
Date: Dec 3, 2019, 15:41 UTC
Message-ID: <xmqqblsp76xs.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20191202232816.GA26892@generichostname>

Denton Liu <liu.denton@gmail.com> writes:

Show 10 quoted lines

>> Especially if it is near the end of the series, just a single step
>> is OK.  But is there anything that is glaringly wrong that needs a
>> reroll?  Or would it be "this is good enough, so let's have them
>> cook in 'next' and graduate to 'master'---further clean-up can be
>> done after all the dust settles"?  I have an impression that we
>> reached the latter by now.
>
> Perhaps the log message could use some improvement to document the
> discussion we had? I don't know if that's worth a reroll, though. Aside
> from that, I agree that it's ready for 'next'.

Sure, let's see what you have in mind.

Thanks for working on this.

$ 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.