[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[paul j3]

Change by paul j3 :


--
nosy: +paul.j3

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[Toshio Kuratomi]

Toshio Kuratomi  added the comment:

PR Opened.

A fix for this should be backported as well.  However, if you decide you don't 
want the refactor backported, you can merely continue to change the condition 
inside of BooleanOptionalAction to repeat all of the same checks as are 
contained in the older versions' ArgumentDefaultsHelpFormatter.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[Toshio Kuratomi]

Change by Toshio Kuratomi :


--
pull_requests: +26275
pull_request: https://github.com/python/cpython/pull/27808

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[Łukasz Langa]

Łukasz Langa  added the comment:

Oh, I forgot to say this explicitly: an important reason why the trivial fix 
was desirable is that it was a clear-cut bugfix, making it easy to backport all 
the way to the 3.9 branch where BooleanOptionalAction was introduced.

Having this in place, your potential PR here might be a cleaner refactor 
without having to worry about backporting. Of course, if we manage to keep it 
small and isolated, we can bring it to 3.9 and 3.10 as well.

--
versions: +Python 3.11 -Python 3.10, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[Łukasz Langa]

Łukasz Langa  added the comment:

Toshio, as I commented on GH-27672, that PR was merged because it's a rather 
trivial improvement over the status quo. I intend to also release this fix in 
3.9.7 on August 30th.

Your idea in this issue to further improve the situation while retaining 
displaying the default value (option 2 described above) is interesting. If you 
prepare a PR we can work with, I'll look into merging that as well.

--
nosy: +lukasz.langa

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[Maximilian Hils]

Change by Maximilian Hils :


--
keywords: +patch
nosy: +mhils
nosy_count: 2.0 -> 3.0
pull_requests: +26259
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/27672

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue44587] argparse BooleanOptionalAction displays default=SUPPRESS unlike other action types

[Ned Deily]

Change by Ned Deily :


--
nosy: +rhettinger
title: BooleanOptionalAction displays default=SUPPRESS unlike other action 
types -> argparse BooleanOptionalAction displays default=SUPPRESS unlike other 
action types

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com