-
Notifications
You must be signed in to change notification settings - Fork 65
starknet_os: move commitment info #10877
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
This stack of pull requests is managed by Graphite. Learn more about stacking. |
eca0c08 to
5ec05e6
Compare
d37258c to
0ad4144
Compare
Yoni-Starkware
left a comment
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.
@Yoni-Starkware made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @avi-starkware, @AvivYossef-starkware, and @noaov1).
crates/starknet_os/src/io/os_input.rs line 46 at r1 (raw file):
/// Contains all commitment information for a block's state trees. pub struct CommitmentInfos {
Now that it's here, add State since there are other commitments in block.
Please add a TODO to use this struct in OsBlockInput.
Suggestion:
/// Contains all commitment information for a block's state trees.
pub struct StateCommitmentInfos {0ad4144 to
4dec46f
Compare
5ec05e6 to
39dc73c
Compare
9263426 to
fac9e2d
Compare
AvivYossef-starkware
left a comment
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.
@AvivYossef-starkware made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @avi-starkware, @noaov1, and @Yoni-Starkware).
crates/starknet_os/src/io/os_input.rs line 46 at r1 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
Now that it's here, add
Statesince there are other commitments in block.
Please add a TODO to use this struct inOsBlockInput.
Done.
meship-starkware
left a comment
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.
@meship-starkware reviewed 2 files and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @avi-starkware, @noaov1, and @Yoni-Starkware).
Yoni-Starkware
left a comment
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.
@Yoni-Starkware reviewed 2 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @avi-starkware and @noaov1).
39dc73c to
eb38642
Compare
fac9e2d to
b50c388
Compare
b50c388 to
07e9212
Compare
AvivYossef-starkware
left a comment
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.
@AvivYossef-starkware reviewed 2 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @avi-starkware and @noaov1).

No description provided.