-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Improve connection reuse test coverage #10949
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
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #10949 +/- ##
=======================================
Coverage 98.77% 98.77%
=======================================
Files 129 129
Lines 39478 39539 +61
Branches 2186 2186
=======================================
+ Hits 38993 39054 +61
Misses 337 337
Partials 148 148
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
CodSpeed Performance ReportMerging #10949 will not alter performanceComparing Summary
|
Backport to 3.12: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 06e3b36 on top of patchback/backports/3.12/06e3b3682c72ffa56b48087cf9b05b3f71f95ab1/pr-10949 Backporting merged PR #10949 into master
🤖 @patchback |
(cherry picked from commit 06e3b36)
I'm planning on removing
.release()
from the examples and the digest middleware. I wanted to make sure everything is working correctly before I proceed with that.