-
Notifications
You must be signed in to change notification settings - Fork 6
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
merge rollup #153
merge rollup #153
Conversation
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.
overall looks like a great start! One question regarding the rollup level
cpp/src/aztec3/circuits/abis/rollup/merge/merge_rollup_public_inputs.hpp
Show resolved
Hide resolved
7c62264
to
d1a4e0f
Compare
d1a4e0f
to
095526f
Compare
02d3751
to
4b6aed4
Compare
fd26d93
to
29da979
Compare
yolo - ignore wasm tests to move ahead and unblock sequencer.
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.
LGTM, made some changes in #166 to make it easier to reuse code from the merge rollup inside the root rollup.
Description
Closes #158 and #159 as well.
Out of scope:
Checklist:
/specs
have been updated.@brief
describing the intended functionality.