workflow-cli icon indicating copy to clipboard operation
workflow-cli copied to clipboard

apps:destroy should have a more explicit help message

Open Overdrivr opened this issue 8 years ago • 5 comments

Running

deis apps:destroy <app-name>

yields the help message because syntax is not correct. The -a flag is missing. However, the help message is not very helpful, it should at least mention this flag.

$ deis apps:destroy italic-nuthatch
Usage: deis apps:destroy [options]

It's not a big deal, but IMO it would improve ergonomy a little bit

Overdrivr avatar Jun 14 '17 05:06 Overdrivr

Is deis apps:destroy --help helpful for you? As a general rule we didn't list options because they're fairly standardized and not necessary. For example, running deis apps:destroy in the app's directory would destroy it.

Joshua-Anderson avatar Jun 14 '17 16:06 Joshua-Anderson

Perhaps a follow-up would be to display the help text instead of a generic usage message. That way it's the best of both worlds: the user gets more helpful usage instructions, and we don't have to individually list each option flag available to each command.

bacongobbler avatar Jun 14 '17 16:06 bacongobbler

I know cobra has a way to change the usage message... Not sure about docopt-go (what workflow-cli uses).

bacongobbler avatar Jun 14 '17 16:06 bacongobbler

@Joshua-Anderson My bad, I thought I was already getting the full help message while in fact it was a stripped down version. I do agree with you @bacongobbler

Overdrivr avatar Jun 15 '17 05:06 Overdrivr

This issue was moved to teamhephy/workflow-cli#4

Cryptophobia avatar Mar 21 '18 14:03 Cryptophobia