-
-
Notifications
You must be signed in to change notification settings - Fork 33.8k
GH-108009: Add clarification of parser and argument defaults in argparse docs #124154
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
CAM-Gerlach
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Stock reminder: You can apply my suggestions with one click using the Commit suggestion button :)
I agree with the change overall, though I left a suggestion to make it substantially shorter, simpler and clearer, which hopefully helps address others' concerns about overcomplicating the docs. Thanks!
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
|
@hpaulj, does this change satisfy your criteria? |
|
Thanks @savannahostrowski for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
… argparse docs (pythonGH-124154) (cherry picked from commit 95f6e12) Co-authored-by: Savannah Ostrowski <savannah@python.org> Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
|
GH-141084 is a backport of this pull request to the 3.14 branch. |
… argparse docs (pythonGH-124154) (cherry picked from commit 95f6e12) Co-authored-by: Savannah Ostrowski <savannah@python.org> Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
|
GH-141085 is a backport of this pull request to the 3.13 branch. |
… argparse docs (python#124154) Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
This PR clarifies the nuances of setting defaults at the parser and argument level using argparse.
📚 Documentation preview 📚: https://cpython-previews--124154.org.readthedocs.build/