bpo-40862: Raise TypeError when const is given to argparse.BooleanOptionalAction by remilapeyre · Pull Request #20623 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.7k
Conversation
the-knights-who-say-ni
added
the
CLA signed
label
bedevere-bot
added
the
awaiting review
label
remilapeyre
mentioned this pull request
hauntsaninja
approved these changes
hauntsaninja
left a comment
hauntsaninja
left a comment
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, lgtm apart from comment nit
Lib/test/test_argparse.py Outdated
bedevere-bot
added
awaiting core review
and removed
awaiting review
labels
Co-authored-by: Shantanu <hauntsaninja@users.noreply.github.com>
rhettinger
self-assigned this
rhettinger
approved these changes
bedevere-bot
added
awaiting merge
and removed
awaiting core review
labels
rhettinger
added
needs backport to 3.9
skip news
labels
rhettinger
merged commit
b084d1b
into
python:master
miss-islington
commented
Jun 5, 2020
miss-islington commented
Contributor
Thanks @remilapeyre for the PR, and @rhettinger for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖
bedevere-bot
removed
the
awaiting merge
label
bedevere-bot
commented
Jun 5, 2020
bedevere-bot commented
GH-20664 is a backport of this pull request to the 3.9 branch.
bedevere-bot
removed
the
needs backport to 3.9
label
rhettinger pushed a commit that referenced this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment