-
Notifications
You must be signed in to change notification settings - Fork 15
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
build: serde transactions #943
build: serde transactions #943
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @AvivYossef-starkware and the rest of your teammates on Graphite |
Benchmark movements: |
455a545
to
2fdcd8d
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## aviv/test_regression_test_utils #943 +/- ##
====================================================================
+ Coverage 62.30% 75.78% +13.48%
====================================================================
Files 127 87 -40
Lines 12974 11090 -1884
Branches 12974 11090 -1884
====================================================================
+ Hits 8083 8405 +322
+ Misses 4474 2273 -2201
+ Partials 417 412 -5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Benchmark movements: |
build: regression test crate
build: blockifier regression test add block info
build: blockifier regression test add get fee token addresses
build: serde transactions
NO
This change is