From: Justin Tobler <jltobler@gmail.com>
Date: Wed, 17 Dec 2025 17:49:59 GMT
Subject: Re: [PATCH v4 0/7] builtin/repo: add object size info to structure output
Message-ID: <4zhiuhpvik5w2vgawepbvqfsfukkixjk7ht6ixvzhjfbu4wtzz@tqljiqujmy47>
In-Reply-To: <aUJVzVp9VB7tDfA-@pks.im>

On 25/12/17 08:03AM, Patrick Steinhardt wrote:
> On Tue, Dec 16, 2025 at 11:38:35AM -0600, Justin Tobler wrote:
> > Changes in V4:
> > - Unmark "byte" string in "t/helper/test-simple-ipc.c" for translation
> >   to avoid conflict with translated plural "byte/bytes" string.
> > - Remove some unnecessary translations and add comments to clarify some
> >   of the added translations.
> > - Some small changes to the tests in patch 7.
> 
> I had a last tiny nit that doesn't warrant a reroll on its own. Other
> than that this series looks great to me now. Thanks!

Junio also had some small comments. I'll go ahead a send another
version. Thanks for the review. :)

-Justin