Re: [PATCH 5/5] completion: drop "stash save" completion altogether
- From
Ben Knoble <ben.knoble@gmail.com>
- Date
- Oct 19, 2026, 20:01 UTC
- Message-ID
- <35E6B235-6370-4F94-A0B1-4B6A941D8C47@gmail.com>
- In-Reply-To
- <atHZ0gV5Pz80l3ho@pks.im>
Show 21 quoted lines
> Le 16 oct. 2026 à 04:01, Patrick Steinhardt <ps@pks.im> a écrit : > > On Mon, Oct 12, 2026 at 01:48:31PM -0400, D. Ben Knoble wrote: >> It's been deprecated since fd2ebf14db (stash: mark "git stash save" >> deprecated in the man page, 2017-10-22) and the previous commit removed >> further documentation suggesting to use it. >> >> Since we recommend push instead, stop completing for "save" and remove a >> special case in completion code. > > This is the only part I'm a bit torn on. I think it's reasonable to not > autocomplete the "save" command, but if the user manually types it, > should we really stop autocompleting its options altogether? > > Maybe the answer is "yes". After all, as you mentioned in the preceding > commit, the command is deprecated since 2017 already! And after almost > 10 years I think it's fair to start giving users more nudges into the > direction of using "push" instead. > > One note though: the command _will_ be completed in the case where you > have GIT_COMPLETION_SHOW_ALL set
I should definitely mention SHOW_ALL; I’m not actually sure what happens for « git stash save --<TAB> » with this patch, so I’ll double-check. I actually wonder if it will complete options.
Would that satisfy you if so ?