-
-
Notifications
You must be signed in to change notification settings - Fork 277
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
Fixed deprecated usage of isAlive() #377
Conversation
Codecov Report
@@ Coverage Diff @@
## master #377 +/- ##
==========================================
- Coverage 94.91% 94.56% -0.36%
==========================================
Files 7 7
Lines 846 846
==========================================
- Hits 803 800 -3
- Misses 43 46 +3
Continue to review full report at Codecov.
|
@gabrielfalcao could we get this merged? |
httpretty throws: - a PendingDeprecationWarning : httpretty/core.py:438: PendingDeprecationWarning: isAlive() is deprecated, use is_alive() instead if t.isAlive(): - a ResourceWarning : httpretty/core.py:485: ResourceWarning: unclosed file <_io.BufferedRandom name=7> self.fd = FakeSockFile() There already are issues and PRs to fix these: - gabrielfalcao/HTTPretty#368 - gabrielfalcao/HTTPretty#377 Until they are fixed I'm suppressing them
@gabrielfalcao any updates on when we can have this patch? |
@kamikaz1k thanks for the reminder <3 |
fixed and released on 0.9.7 |
it bumps the httpretty version to 0.9.7, which fixes the warning on isAlive. ref: gabrielfalcao/HTTPretty#377
isAlive() is deprecated, use is_alive() instead