-
Notifications
You must be signed in to change notification settings - Fork 73
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
feat: interop invariant checks #58
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @tremarkley and the rest of your teammates on Graphite |
71aeb12
to
bbe0bf1
Compare
d4f48f4
to
85e420b
Compare
85e420b
to
88441eb
Compare
8106add
to
1c01ca2
Compare
1c01ca2
to
d518988
Compare
8156782
to
975a0be
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.
Looks good! Left some feedback
Some of the interop checks can be pulled into utility functions but we can iterate on that instead of trying to get it right first
544b9cf
to
9913d6c
Compare
9913d6c
to
eaee40b
Compare
Closes: #19
Closes: #18
Adds interop invariant checks to the opsimulator, and will reject transactions that do not meet the interop invariants. The interop invariant checks are in line with the interop spec: https://specs.optimism.io/interop/overview.html