Subject: Re: [PATCH v4 1/1] [PATCH] t9117: prefer test_path_* helper functions

From: Junio C Hamano <[email-masked]>
Date: Mon, 04 Mar 2024 09:22:32 -0800

shejialuo <[email-masked]> writes:

> test -(e|d) does not provide a nice error message when we hit test
> failures, so use test_path_exists, test_path_is_dir instead.

OK.

>
> Signed-off-by: shejialuo <[email-masked]>
> ---

Just for the next single-patch topic you'd work on, here below the
three-dash line is where you may mention what's different between
the previous iteration and this one, if you wanted to, instead of
having a separate cover-letter message.

>  t/t9117-git-svn-init-clone.sh | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)

The patch looks good to me.  Thanks (and thanks for all the
reviewers of the previous rounds).