Which abra commands should be completed by which autocomplete function? #87

Closed
opened 2021-09-07 13:42:49 +00:00 by knoflook · 2 comments
Owner

@decentral1se:

Added bonus request: seems like we can BashComplete: func(c *cli.Context) { ... }? for abra app $tab/abra recipe $tab? (#83)

I'm not sure if the title makes sense but I can't think of a way to write this sentence in a non-awkward way. I'm gonna write 4 autocomplete functions for now that will return:

  • all apps
  • deployed apps
  • undeployed apps
  • all recipes

now, this raises some questions such as:
should abra app rm be completed by all apps or deployed apps?? We don't want to remove an undeployed app but maybe the user wants to have all of them listed?

@decentral1se: >Added bonus request: seems like we can BashComplete: func(c *cli.Context) { ... }? for abra app $tab/abra recipe $tab? (#83) I'm not sure if the title makes sense but I can't think of a way to write this sentence in a non-awkward way. I'm gonna write 4 autocomplete functions for now that will return: * all apps * deployed apps * undeployed apps * all recipes now, this raises some questions such as: should `abra app rm` be completed by all apps or deployed apps?? We don't want to remove an undeployed app but maybe the user wants to have all of them listed?
knoflook added the
question
design
labels 2021-09-07 13:42:49 +00:00
decentral1se added this to the (deleted) milestone 2021-09-07 13:43:12 +00:00
Author
Owner

It's also important to remember that getting deployed/undeployed apps might take some time... maybe it's best to just stay with all apps and all recipes?

edit: it takes a couple of seconds to load deployed apps. I don't know if it's worth it

It's also important to remember that getting deployed/undeployed apps might take some time... maybe it's best to just stay with all apps and all recipes? edit: it takes a couple of seconds to load deployed apps. I don't know if it's worth it
Owner

yeah fuck it, just list all apps without doing the lookup then, sounds good!

yeah fuck it, just list all apps without doing the lookup then, sounds good!
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#87
No description provided.