-
-
Notifications
You must be signed in to change notification settings - Fork 292
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
Don't track unstable LinkedList benchmark #3809
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3809 +/- ##
=======================================
Coverage 36.82% 36.82%
=======================================
Files 324 324
Lines 8870 8870
Branches 1385 1385
=======================================
Hits 3266 3266
Misses 5461 5461
Partials 143 143 |
Performance Report✔️ no performance regression detected Full benchmark results
|
@@ -2363,7 +2363,7 @@ | |||
resolved "https://registry.yarnpkg.com/@types/json5/-/json5-0.0.29.tgz#ee28707ae94e11d2b827bcbe5270bcea7f3e71ee" | |||
integrity sha1-7ihweulOEdK4J7y+UnC86n8+ce4= | |||
|
|||
"@types/jwt-simple@^0.5.33": | |||
"@types/[email protected]": |
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.
not sure why this is changed modified
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.
This PR #3777 did not update the yarn.lock after requiring a different version. Running yarn causes this diff
Motivation
We got the results of the data-structure performance, which is already implemented. Disable since it's failing CI builds
Description
Don't track unstable LinkedList benchmark