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

Replace - with _ when user passes "extra_arguments" #164

Closed
lorengordon opened this issue Mar 6, 2017 · 0 comments
Closed

Replace - with _ when user passes "extra_arguments" #164

lorengordon opened this issue Mar 6, 2017 · 0 comments

Comments

@lorengordon
Copy link
Member

When using an explicity-defined cli opt, argparse replaces - with _ by default. We are also accepting arbitrary cli opts into the extra_arguments list, which argparse does not munge. For the user, this means currently they would need to do this:

watchmaker --content_source <url.zip>

which doesn't match the structure of every other cli opt. Instead, we should munge extra_arguments to support the same structure as regular cli opts:

watchmaker --content-source <url.zip>
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

1 participant