-
Notifications
You must be signed in to change notification settings - Fork 56
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
required
options don't get noted as required in the generated manpage/docs
#82
Comments
Required optional arguments are a contradiction in the terms. You should rather use positional arguments for required arguments. |
After two or three, positional arguments are difficult to read. We have many required args in this application. We also want to disambiguate some arguments (e.g. the type of |
My guess is that you'll soon end up with a configuration file or dsl. In any case ded2040 now indicates required optional in the documentation of options. As far as synopses are concerned this should be solved in conjunction with #42 so I'm tracking this there. |
Something we're desperately trying to avoid, but ending up with a CLI DSL anyway… there is no escaping that. Thank you for the addition, and looking forward to seeing what comes out of that issue as well. |
Note that I'm not sure this will make it in the next release, but you can always define the |
e.g. in hammerlab/epidisco
And yet it is rendered as
and
The text was updated successfully, but these errors were encountered: