search

deis / workflow-cli

The CLI for Deis Workflow
http://deis.com
MIT License
31 stars 43 forks source link

apps:destroy should have a more explicit help message #310

Open Overdrivr opened 7 years ago

Overdrivr commented 7 years ago

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

Joshua-Anderson commented 7 years ago

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.

bacongobbler commented 7 years ago

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 commented 7 years ago

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

Overdrivr commented 7 years ago

@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

Cryptophobia commented 6 years ago

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