-
Notifications
You must be signed in to change notification settings - Fork 281
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
fix out of range access, minor performance improvement (0.27-maintenance backport) #1925
fix out of range access, minor performance improvement (0.27-maintenance backport) #1925
Conversation
Codecov Report
@@ Coverage Diff @@
## 0.27-maintenance #1925 +/- ##
=================================================
Coverage 46.30% 46.30%
=================================================
Files 146 146
Lines 22948 22948
Branches 11781 11781
=================================================
Hits 10626 10626
Misses 6707 6707
Partials 5615 5615
Continue to review full report at Codecov.
|
I don't understand why the codecov check is failing. I am very sure that the new tests cover every branch of the new code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kevinbackhouse As always, thank you for working on this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kevinbackhouse As always, thank you for working on this.
This is a backport of #1918. Mergify didn't work.