Skip to content

Use Warning classes not ignored by default filters #71

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

Merged
merged 1 commit into from
Sep 16, 2022

Conversation

ajnelson-nist
Copy link
Member

This patch addresses a now-resolved curiosity where DeprecationWarnings were not being visibly raised when intentionally triggered in an application. They turn out to be ignored by default.

UserWarnings were also not displaying by default, so this patch designates some of the erroneous behaviors not intended to cause halts as RuntimeWarnings.

This patch also adds the expected environment variable name to a docstring visible in the command-line help() interface.

References:

This patch addresses a now-resolved curiosity where
`DeprecationWarning`s were not being visibly raised when intentionally
triggered in an application.  They turn out to be ignored by default.

`UserWarning`s were also not displaying by default, so this patch
designates some of the erroneous behaviors not intended to cause halts
as `RuntimeWarning`s.

This patch also adds the expected environment variable name to a
docstring visible in the command-line help() interface.

References:
* https://docs.python.org/3/library/warnings.html

Signed-off-by: Alex Nelson <alexander.nelson@nist.gov>
@ajnelson-nist ajnelson-nist force-pushed the fix_deprecation_warning_style branch from c27ac59 to a0b2d1e Compare September 16, 2022 15:22
@ajnelson-nist ajnelson-nist marked this pull request as ready for review September 16, 2022 15:59
@kchason kchason merged commit 0173443 into develop Sep 16, 2022
@kchason kchason deleted the fix_deprecation_warning_style branch September 16, 2022 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants