-
Notifications
You must be signed in to change notification settings - Fork 13
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
Rollups meta step #82
Conversation
0fb3c08
to
17c170d
Compare
17c170d
to
2b92765
Compare
d72610f
to
9a9b2bb
Compare
9a9b2bb
to
0c46830
Compare
An rollups integration test is added with a Lua orchestrator script which does the following:
Although the code may seem working fine, I still think we need to:
To run the integration test with the following steps:
|
0c46830
to
b5ae814
Compare
b5ae814
to
5820fd1
Compare
96639e0
to
0efe627
Compare
0efe627
to
e64580b
Compare
e64580b
to
ea06864
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.
Great work!
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!
Depends on cartesi/machine-solidity-step#74 to pass the tests