Skip to content

Add builtin BooleanOptionalAction Action#12

Merged
fmenabe merged 1 commit intofmenabe:masterfrom
mckaymatt:add-BooleanOptionalAction-action
May 20, 2022
Merged

Add builtin BooleanOptionalAction Action#12
fmenabe merged 1 commit intofmenabe:masterfrom
mckaymatt:add-BooleanOptionalAction-action

Conversation

@mckaymatt
Copy link
Contributor

Argparse added a new builtin Action class and I figured it would make sense to add that as a default. Let me know what you think.
https://docs.python.org/3/library/argparse.html?highlight=argparse#action

@mckaymatt mckaymatt marked this pull request as ready for review May 19, 2022 18:38
@mckaymatt
Copy link
Contributor Author

@fmenabe

@fmenabe fmenabe merged commit 7724bc1 into fmenabe:master May 20, 2022
@fmenabe
Copy link
Owner

fmenabe commented May 20, 2022

Thanks for the PR @mckaymatt! Looks good to me.

I just released the new version on Pypi and rebuild the docs.

@mckaymatt mckaymatt deleted the add-BooleanOptionalAction-action branch May 20, 2022 15:41
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.

2 participants