Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
Overdrivr opened this issue Jun 14, 2017 · 5 comments
Open

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

Overdrivr opened this issue Jun 14, 2017 · 5 comments

Comments

@Overdrivr
Copy link

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
Copy link
Contributor

Joshua-Anderson commented Jun 14, 2017

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
Copy link
Member

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
Copy link
Member

bacongobbler commented Jun 14, 2017

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

@Overdrivr
Copy link
Author

@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
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants