-
Notifications
You must be signed in to change notification settings - Fork 124
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
Error: warnings has no attribute DeprecationWarning #75
Comments
@nicoddemus wow yeah not sure how that slipped through. |
@nicoddemus you ok if I just fix this in #72 or should I do a separate PR? |
I'm running pytest's test suite right now and fixing the |
You replied a few seconds before me hehehe. If you want to take a stab please go ahead, I just think we should do a separate PR because we also need to include a proper test for it. |
@nicoddemus ok great I'm cool with you taking it ;) |
OK sounds good. |
While testing
pluggy
off the branch from #72, I came across this error:pluggy/pluggy/__init__.py
Lines 656 to 661 in 557cd3f
And indeed
warnings
module doesn't have aDeprecationWarning
.We should add a test to catch this warnings (and any others we might have).
The text was updated successfully, but these errors were encountered: