Re: [PATCH v3 2/3] sparse-checkout: custom tab completion

From: Lessley Dennington <lessleydennington@gmail.com>
Date: Jan 18, 2022, 17:59 UTC
Message-ID: <14ada2ac-9a10-9653-6ced-9cfc675b9419@gmail.com>
In-Reply-To: <CABPp-BGR+UrW5ej-8+XXHPkjMfFgURycd9rWC+2awUvYcr_PXA@mail.gmail.com>

On 1/15/22 5:03 PM, Elijah Newren wrote:

Show 13 quoted lines

> On Sat, Jan 15, 2022 at 1:57 AM SZEDER Gábor <szeder.dev@gmail.com> wrote:
>>
>> On Mon, Jan 10, 2022 at 06:59:51PM +0000, Lessley Dennington via GitGitGadget wrote:
>>> Subject: Re: [PATCH v3 2/3] sparse-checkout: custom tab completion
>>
>> None of these patches touch sparse-checkout, but only the completion
>> script and its tests.  Therefore "completion:" would be a better
>> matching area prefix.
> 
> Thanks for the detailed feedback and guidance in your review.  Very
> helpful.  I'll omit quoting most of it here, but I do want to comment
> on the point about directories.
>

I second this - am planning to apply all feedback (with the exception of completing on both files and directories) in v4 of this series.