-
Notifications
You must be signed in to change notification settings - Fork 201
Rename plasma to altDA #439
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
refcell
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.
Rust changes look good.
7fc039e to
a5c069c
Compare
geoknee
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.
The rollup config diff tool is failing, and nowadays that failure is not expected/acceptable in general.
I think the problem is that the tool cannot generate configs on main, since the tool only checks out the superchain folder from main and retains the rest of the code from the PR branch. That might be an easy fix -- @bitwiseguy do you think you could look into it?
d8880da to
c27b229
Compare
|
Detected nonzero diff in rollup config. This PR modifies the configuration of at least one chain in the registry, which may or may not have been intended. You can see the detail here. |
Fixes #429
Complementary monorepo pr: ethereum-optimism/optimism#11370