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.

Sign up for GitHub

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

Jump to bottom

Show usage instructions when you give bad arguments to replicate run #159

Closed
andreasjansson opened this issue Sep 30, 2020 · 0 comments · Fixed by #165
Closed

Show usage instructions when you give bad arguments to replicate run #159

andreasjansson opened this issue Sep 30, 2020 · 0 comments · Fixed by #165
Labels
priority/medium type/bug Something isn't working

Comments

@andreasjansson
Copy link
Member

If you get the CLI args wrong you get the not very helpful message:

$ ~/r8/replicate/python/build/bin/replicate run -m /tmp/book-rec/data:/tmp/book-rec/data
═══╡ requires at least 1 arg(s), only received 0
@andreasjansson andreasjansson added type/bug Something isn't working priority/medium labels Sep 30, 2020
bfirsh added a commit that referenced this issue Sep 30, 2020
@bfirsh
Print usage for CLI parsing errors
4a70740 
Without SilenceUsage, errors in RunE also produce usage, so we have
to make our own wrapper that prints errors without usage.

See also:
spf13/cobra#340
spf13/cobra#914

Closes #159
@bfirsh bfirsh mentioned this issue Sep 30, 2020
Merged
bfirsh added a commit that referenced this issue Oct 1, 2020
@bfirsh
Print usage for CLI parsing errors
24f6021 
Without SilenceUsage, errors in RunE also produce usage, so we have
to make our own wrapper that prints errors without usage.

See also:
spf13/cobra#340
spf13/cobra#914

Closes #159
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority/medium type/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Print usage for CLI parsing errors replicate/keepsake
1 participant
@andreasjansson