Re: [PATCH v2 0/2] test: cleanups of aggregate-results.sh

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 10, 2023, 19:25 UTC
Message-ID: <xmqqa60k5ryc.fsf@gitster.g>
In-Reply-To: <20230309173032.959968-1-felipe.contreras@gmail.com>

Felipe Contreras <felipe.contreras@gmail.com> writes:

Show 15 quoted lines

> Basically $title.
>
> Since v1 I'm now passing the directory, instead of the file pattern as
> Ævar suggested.
>
> Also, while at it, let's silence the print of the command.
>
> Felipe Contreras (2):
>   test: simplify counts aggregation
>   test: don't print aggregate-results command
>
>  contrib/subtree/t/Makefile | 4 +---
>  t/Makefile                 | 4 +---
>  t/aggregate-results.sh     | 2 +-
>  3 files changed, 3 insertions(+), 7 deletions(-)

These two look sensible. Any comments on them by reviewers on the initial round?

Will queue.  Thanks, all.