From: Taylor Blau <me@ttaylorr.com>
Date: Sat, 01 Apr 2023 00:05:43 GMT
Subject: Re: [PATCH 2/6] test: avoid `stat` variable
Message-ID: <ZCd1Vyr+l9w/NmZ+@nand.local>
In-Reply-To: <20230328173932.3614601-3-felipe.contreras@gmail.com>

On Tue, Mar 28, 2023 at 11:39:28AM -0600, Felipe Contreras wrote:
> ---
>  t/t0001-init.sh                | 4 ++--
>  t/t3432-rebase-fast-forward.sh | 4 ++--
>  t/t4013-diff-various.sh        | 6 +++---
>  t/t5512-ls-remote.sh           | 4 ++--
>  t/t5516-fetch-push.sh          | 8 ++++----
>  t/test-lib.sh                  | 4 ++--
>  6 files changed, 15 insertions(+), 15 deletions(-)

Maybe I'm misreading your patch or its subject line, but all of these
hunks convert from something (usually "status") _to_ "stat". Is the
point to avoid "status", not "stat"? Or the other way around?

Thanks,
Taylor