-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
GH-101599: Update docs to remove redundant option in argparse tutorial 8000 #124025
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
Conversation
@hugovk or @serhiy-storchaka, perhaps one of you could have a look since you were both active on the PR (or on the issue #101599 (comment)) that changed this behaviour? |
Nice catch, thank you! |
Thanks @savannahostrowski for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12. |
Thanks @savannahostrowski for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13. |
…utorial (pythonGH-124025) (cherry picked from commit e5b0185) Co-authored-by: Savannah Ostrowski <savannahostrowski@gmail.com>
…utorial (pythonGH-124025) (cherry picked from commit e5b0185) Co-authored-by: Savannah Ostrowski <savannahostrowski@gmail.com>
GH-124036 is a backport of this pull request to the 3.12 branch. |
GH-124037 is a backport of this pull request to the 3.13 branch. |
The change went into 3.13 (#103372), so no need to backport to 3.12. |
This PR updates docs to reflect updates made in #103372. I discovered this as a duplicate while closing out on #77570. I've also skimmed through the argparse docs and tutorial for other instances where this may need to be updated. This appears to be the only instance that needed to be updated.
argparse
Prints options per flag name when only once is necessary #101599📚 Documentation preview 📚: https://cpython-previews--124025.org.readthedocs.build/