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
There are many problems with exit status people encounter using argparse (and some of them are open, e.g. #85427).
It is reasonable and consistent with other code that it raises SystemExit (in case of unrecognized arguments).
However, this is not flexible and requires pretty much work from users if they want to catch this error, and not exit (there are many alternatives (which in itself contradicts the Zen of Python), the default proposes to subclass the ArgumentParser).
Why can't argparse just subclass SystemExit? This is usually a good guideline for a module to create its own exception classes.
Also, all three argparse exceptions could derive from one argparse exception class (to make their interception easier) (in the title I suggested ArgparseError, but a better name might be found). However, this is not so crucial as this SystemExit: I run a big function in one place of which there is parse_args; I don't want to attribute all SystemExit-s to argparse when I catch exceptions from that function (for now I had to create my own subclass of SystemExit raised from the line with parse_args).
And (should it be a separate feature request?) could there be a subsection dedicated to exceptions in the argparse module documentation? I had to read Python help to understand the initialisation of ArgumentError.
If argparse could guarantee the types of exceptions that it can raise, that would be ideal.
There are many problems with exit status people encounter using
argparse
(and some of them are open, e.g. #85427).It is reasonable and consistent with other code that it raises
SystemExit
(in case of unrecognized arguments).However, this is not flexible and requires pretty much work from users if they want to catch this error, and not exit (there are many alternatives (which in itself contradicts the Zen of Python), the default proposes to subclass the ArgumentParser).
Why can't
argparse
just subclassSystemExit
? This is usually a good guideline for a module to create its own exception classes.Also, all three
argparse
exceptions could derive from oneargparse
exception class (to make their interception easier) (in the title I suggestedArgparseError
, but a better name might be found). However, this is not so crucial as thisSystemExit
: I run a big function in one place of which there isparse_args
; I don't want to attribute all SystemExit-s toargparse
when I catch exceptions from that function (for now I had to create my own subclass ofSystemExit
raised from the line withparse_args
).And (should it be a separate feature request?) could there be a subsection dedicated to exceptions in the
argparse
module documentation? I had to read Python help to understand the initialisation ofArgumentError
.If argparse could guarantee the types of exceptions that it can raise, that would be ideal.
Originally posted by @ericvsmith in #83514 (comment)
The text was updated successfully, but these errors were encountered: