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

allow boolean switch with '--enable-foo=true/false' #318

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rtgiskard
Copy link

This is a try to implement the #313, however this make flag() apply nargs(0,1), and break assumption for some of the test.

For example: program.parse_args({"./test", "--enable-foo", "dump"}), with --enable-foo set with flag(), the positional argument dump will be consumed by the flag and cause trouble.

And I'm not sure, this seems to be in consistent with the original design, there may be better solutions.

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

Successfully merging this pull request may close these issues.

1 participant