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

[CWL] ignore const #50

Open
3 tasks
mr-c opened this issue Apr 16, 2017 · 0 comments
Open
3 tasks

[CWL] ignore const #50

mr-c opened this issue Apr 16, 2017 · 0 comments
Labels

Comments

@mr-c
Copy link
Collaborator

mr-c commented Apr 16, 2017

From https://github.com/erasche/argparse2tool/blob/master/README.rst#limitations

const argument of add_argument(). All constants must be specified in job files.

My reading of https://docs.python.org/2/library/argparse.html#const is that const is an internal implementation detail -- I suspect that it can be safely ignored for the purpose of generating command line invocations.

  • find real world usage of const
  • test ignoring const
  • update readme and perhaps add a comment in the code if this is indeed ignorable
@mr-c mr-c added the CWL label Apr 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant