-
Notifications
You must be signed in to change notification settings - Fork 239
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
eager length checks #610
eager length checks #610
Conversation
CodSpeed Performance ReportMerging #610 Summary
Benchmarks breakdown
|
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## main #610 +/- ##
==========================================
- Coverage 94.25% 93.99% -0.26%
==========================================
Files 98 98
Lines 13188 13281 +93
Branches 25 25
==========================================
+ Hits 12430 12484 +54
- Misses 752 791 +39
Partials 6 6
... and 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
This reverts commit 4433435.
WIP alternative take on #602 seems to have minimal performance impact, apart from copying
list[Any]
etc.