-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
MAINT: Remove pytest.warns in tests #19341
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
Conversation
I added the linting but will push fix in second commit once I confirm linting works. |
@gfyoung this needs to fix the test, no? ) (or maybe you are just testing if this will fail)
|
@jreback : Yes, that's the plan 😉 You can see that it worked. https://travis-ci.org/pandas-dev/pandas/jobs/331740919#L3018 |
Per discussion in pandas-devgh-18258, we are prohibiting its use in tests, at least for the time being.
The lint correctly fails on this line.
499d330
to
09097c8
Compare
Codecov Report
@@ Coverage Diff @@
## master #19341 +/- ##
==========================================
- Coverage 91.57% 91.54% -0.03%
==========================================
Files 150 150
Lines 48702 48702
==========================================
- Hits 44597 44585 -12
- Misses 4105 4117 +12
Continue to review full report at Codecov.
|
@jreback : Lint error has been patched as planned. PTAL. |
thanks @gfyoung I would not necessarily be averse to changing to use |
The main difference (I think) is that ours has |
Per discussion in #18258, we are prohibiting its use in tests, at least for the time being.