-
Notifications
You must be signed in to change notification settings - Fork 22
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
refactor(fee): block context creator may fail #507
refactor(fee): block context creator may fail #507
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @nimrod-starkware and the rest of your teammates on Graphite |
d89e3fd
to
4668bc1
Compare
3f4a277
to
d772755
Compare
4668bc1
to
a9bca83
Compare
d772755
to
5c10c7b
Compare
a9bca83
to
0edc05f
Compare
5c10c7b
to
cbb4e6a
Compare
0edc05f
to
162f6e2
Compare
cbb4e6a
to
c9aa976
Compare
162f6e2
to
61d58fd
Compare
c9aa976
to
ad9d50c
Compare
61d58fd
to
3fafce9
Compare
ad9d50c
to
72bc281
Compare
3fafce9
to
b6769b7
Compare
72bc281
to
cdb9892
Compare
b6769b7
to
dc03a97
Compare
cdb9892
to
0397362
Compare
dc03a97
to
57766e2
Compare
0d6e837
to
f728fc1
Compare
1802aaa
to
c9e4d63
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #507 +/- ##
==========================================
- Coverage 76.63% 76.58% -0.05%
==========================================
Files 351 351
Lines 37092 37093 +1
Branches 37092 37093 +1
==========================================
- Hits 28424 28408 -16
- Misses 6369 6374 +5
- Partials 2299 2311 +12 ☔ View full report in Codecov by Sentry. |
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.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
f728fc1
to
669b111
Compare
c9e4d63
to
1c1afcd
Compare
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.
Reviewed 3 of 3 files at r8, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
669b111
to
8b53820
Compare
1c1afcd
to
18eb487
Compare
8b53820
to
18bf0a3
Compare
18eb487
to
19c7557
Compare
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.
Reviewed 1 of 1 files at r9, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
18bf0a3
to
a658b50
Compare
19c7557
to
a21e7a7
Compare
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.
python side
Reviewable status: 12 of 13 files reviewed, all discussions resolved (waiting on @dorimedini-starkware and @TzahiTaub)
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.
Reviewed 1 of 1 files at r10, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
Merge activity
|
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.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
a21e7a7
to
10d32c9
Compare
This change is