Subject: Re: [PATCH v2] describe: enable sparse index for describe

From: Junio C Hamano <[email-masked]>
Date: Wed, 29 Mar 2023 11:27:41 -0700

Victoria Dye <[email-masked]> writes:

>> diff --git a/t/t6121-describe-sparse.sh b/t/t6121-describe-sparse.sh
>> new file mode 100755
>> index 00000000000..ce53603c387
>> --- /dev/null
>> +++ b/t/t6121-describe-sparse.sh
>
> Is there a specific reason you've created a new test file instead of adding
> the tests to 't1092-sparse-checkout-compatibility.sh'? Historically, ...
> ... 't1092' also uses a more "interesting" test repo &
> includes comparison functions for full checkout/sparse-checkout/sparse
> index, so the tests added for 'git describe' can be more thorough.

Ahh...  I mentioned t6120 in my response, but t1092 does sound like
a lot more appropriate place for adding tests for these.

Thanks for pointing it out.