On Mon, Feb 7, 2022 at 9:31 AM Lessley Dennington via GitGitGadget <gitgitgadget@xxxxxxxxx> wrote: > > This change updates custom tab completion for the sparse-checkout command. > Specifically, it corrects the following issues with the current method: > > 1. git sparse-checkout <TAB> results in an incomplete list of subcommands > (it is missing reapply and add). > 2. Options for subcommands are not tab-completable. > 3. git sparse-checkout set <TAB> and git sparse-checkout add <TAB> show > both file names and directory names. While this may be a less surprising > behavior for non-cone mode, we want to only show directories in cone > mode. > > The first commit in this series is an intermediate step that fixes issues 1 > and 2 above and introduces a simple fix for issue 3 with some performance > and unusual character-related caveats. The next commit adds a new > __gitcomp_directories method that fixes the performance-related caveat from > the first commit by completing just a single level of directories. The final > commit modifies __gitcomp_directories to handle unusual characters in > directory names. > > > Changes since V6 > ================ > > * Split tests for unusual characters so that spaces and accents are > verified on Windows (in addition to macOS and Linux). Tabs and > backslashes are still excluded on Windows (details on why are included in > the third commit message). Thanks for doing that. I think this series is ready for next. (It already has my Reviewed-by, so no need for me to re-give it.)
