-
Notifications
You must be signed in to change notification settings - Fork 714
Update verifybench test data #2453
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
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2453 +/- ##
==========================================
- Coverage 95.54% 95.26% -0.28%
==========================================
Files 95 95
Lines 22892 21256 -1636
==========================================
- Hits 21872 20250 -1622
+ Misses 1020 1006 -14 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Cherry-pick @cpu's check external types upgrade? Maybe add a short README to the testdata dir and/or some comments to the verifybench module about this update process? |
This was four years old, and the servo tests required a now-removed root. The process for this was: - delete all the old data - run admin/capture-certdata - adjust the time in verifybench.rs, and adjust chains that changed in length Document that in a head comment in verifybench.rs
f9f67e3
to
fa0440b
Compare
Matches the nightly used by `cargo-check-external-types` v0.2.0: https://github.com/awslabs/cargo-check-external-types/releases/tag/v0.2.0
Done both of these. |
This was four years old, and the servo tests required a now-removed root.
The process for this was: