feat: consolidate CairoRunner ctors#2368
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. |
8545c3d to
585ec3f
Compare
4fea4f5 to
6222559
Compare
|
Benchmark Results for unmodified programs 🚀
|
6222559 to
38a7083
Compare
8eb0e9e to
fb62714
Compare
38a7083 to
843862d
Compare
fb62714 to
4c088b2
Compare
843862d to
ab535f6
Compare
4c088b2 to
4e722af
Compare
839b966 to
5ef6991
Compare
861ecd7 to
6325b49
Compare
5ef6991 to
1c0aee2
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 03-21-feat_create_and_use_cairorunconfig_as_ctor_arg_for_cairorunner_ctors #2368 +/- ##
===========================================================================================================
Coverage 96.19% 96.19%
===========================================================================================================
Files 105 105
Lines 37831 37810 -21
===========================================================================================================
- Hits 36391 36372 -19
+ Misses 1440 1438 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
6325b49 to
fae8716
Compare
1c0aee2 to
e0b400b
Compare
fae8716 to
b08d653
Compare
80ed22a to
73efd57
Compare
e2e79a1 to
6b7ceae
Compare
73efd57 to
5cb8404
Compare
YairVaknin-starkware
left a comment
There was a problem hiding this comment.
@YairVaknin-starkware reviewed 11 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on Yael-Starkware).
5cb8404 to
3212747
Compare
0182942 to
0d461e8
Compare
3212747 to
0de1656
Compare
0d461e8 to
d7f2194
Compare
3fa2fba to
e1c6c51
Compare
d7f2194 to
cb93e95
Compare
cb93e95 to
f004484
Compare
YairVaknin-starkware
left a comment
There was a problem hiding this comment.
@YairVaknin-starkware reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on Yael-Starkware).

TITLE
Description
Description of the pull request changes and motivation.
Checklist
This change is