You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Suggestion for argparse: raise a warning on type=bool #126676
while we may not want to actually change the behavior (given existing code on the wild might use it, as mentioned in previous discussions), what about raise a warning? Or is that also too intrusive?
I'd probably be more inclined to document recipes/patterns to achieve the desired result if we need additional materials here since we have already documented that bool() is not recommended as a type converter.
The bool() function is not recommended as a type converter. All it does is convert empty strings to False and non-empty strings to True. This is usually not what is desired.
My concern is that adding a warning here could still break tests, etc.
Activity
savannahostrowski commentedon Nov 17, 2024
I'd probably be more inclined to document recipes/patterns to achieve the desired result if we need additional materials here since we have already documented that
bool()is not recommended as a type converter.My concern is that adding a warning here could still break tests, etc.
pythongh-126676: Expand argparse docs for type=bool with warning and …
gh-126676: Expand argparse docs for type=bool with warning and altern…
pythongh-126676: Expand argparse docs for type=bool with warning and …
pythongh-126676: Expand argparse docs for type=bool with warning and …
[3.13] gh-126676: Expand argparse docs for type=bool with warning and…
[3.14] gh-126676: Expand argparse docs for type=bool with warning and…
pythongh-126676: Expand argparse docs for type=bool with warning and …