Skip to content

Use Warning classes not ignored by default filters - #71

Merged
kchason merged 1 commit into
developfrom
fix_deprecation_warning_style
Sep 16, 2022
Merged

Use Warning classes not ignored by default filters#71
kchason merged 1 commit into
developfrom
fix_deprecation_warning_style

Conversation

@ajnelson-nist

Copy link
Copy Markdown
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-nistforce-pushed the fix_deprecation_warning_style branch from c27ac59 to a0b2d1eCompareSeptember 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 developSep 16, 2022
@kchason
kchason deleted the fix_deprecation_warning_style branch September 16, 2022 16:01
Sign up for freeto 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

@ajnelson-nist@kchason