git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Add shell completion for git remote rm

From
KSKeith Smiley <k@keith.so>
Date
Jul 16, 2018, 13:12 UTC
Message-ID
<86CBC1AA-2AE2-4C58-A7BC-A6AD3A3B822D@keith.so>
In-Reply-To
<CACsJy8C0f43cc5TuybEo5wmem9MSdTO+-0tF3c9k4siN1UgwwQ@mail.gmail.com>
Since there hasn't been any movement on the alternative solutions mentioned here, would it be reasonable to accept this patch in the meantime?

-- Keith Smiley

Show 28 quoted lines
> On Jan 17, 2018, at 01:48, Duy Nguyen <pclouds@gmail.com> wrote:
> 
>> On Wed, Jan 17, 2018 at 1:17 PM, Kevin Daudt <me@ikke.info> wrote:
>>> On Wed, Jan 17, 2018 at 07:44:19AM +0700, Duy Nguyen wrote:
>>> PS. This also makes me thing about supporting subcommand aliases, so
>>> that people can add back 'git remote rm' if they like (or something
>>> like 'git r rm' when they alias 'remote' as well). Which is probably a
>>> good thing to do. Long command names are fine when you have completion
>>> support, they are a pain to type otherwise.
>>> 
>> 
>> Couldn't they just create an alias like git rrm then, if they use it so
>> often that it becomes an issue?
> 
> They could. The only exception that may not work is if they want to
> insert some options between "r" and "rm". Sometimes option positions
> matter. Anyway this is just thinking out loud, maybe we don't really
> need it until people scream about it with a valid use case
> 
>> Having another layer of aliases does create a lot of complexity.
> 
> Yes. It's partly the reason I wanted this actually ;-) Many commands
> have gained subcommands nowadays but there's no shared infrastructure
> for managing these subcommands. By adding something that works across
> the board at subcommand level I'm forced to "fix" this (or probably
> never get to do the sub-aliasing because this "fix" takes forever).
> -- 
> Duy
Previous: Duy NguyenNext: Junio C Hamano
Message 19 of 23 in “Add shell completion for git remote rm”
  1. Add shell completion for git remote rmKeith Smiley, Dec 29, 2017
  2. Todd ZullingerDec 29, 2017
  3. Keith SmileyDec 29, 2017
  4. Todd ZullingerDec 29, 2017
  5. Keith SmileyDec 29, 2017
  6. SZEDER GáborDec 29, 2017
  7. Keith SmileyDec 29, 2017
  8. Ævar Arnfjörð BjarmasonDec 29, 2017
  9. Todd ZullingerDec 30, 2017
  10. Ævar Arnfjörð BjarmasonDec 30, 2017
  11. Keith SmileyDec 31, 2017
  12. Junio C HamanoJan 3, 2018
  13. Keith SmileyJan 15, 2018
  14. Duy NguyenJan 16, 2018
  15. Junio C HamanoJan 16, 2018
  16. Duy NguyenJan 17, 2018
  17. Kevin DaudtJan 17, 2018
  18. Duy NguyenJan 17, 2018
  19. Keith SmileyJul 16, 2018
  20. Junio C HamanoJan 17, 2018
  21. Duy NguyenJan 1, 2018
  22. Kevin DaudtDec 29, 2017
  23. Keith SmileyDec 29, 2017

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.