-
Notifications
You must be signed in to change notification settings - Fork 301
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
tests: cleanup warning handling #2834
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2834 +/- ##
=======================================
Coverage 65.32% 65.32%
=======================================
Files 209 209
Lines 28210 28210
=======================================
Hits 18429 18429
Misses 9781 9781 |
@jenshnielsen Should we try to land this before 0.26.0 release? I can take a look at the errors for python 3.9 |
@trevormorgan We could at least land the improvements to the tests to use the correct context managers. I think there is a test within the monitor module that needs significant work on all versions of python so it may take some time to get all warnings enabled |
300f3f7
to
dc7227b
Compare
dc7227b
to
51bc046
Compare
@trevormorgan and @astafan8 Retargeted this to just cleanup some warning handling in tests so this should be ready to land now |
This should eventually help us get to a point where warnings can become errors for testing