-
Notifications
You must be signed in to change notification settings - Fork 99
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
Cleanup: unify typing of state variables in StakeRegistryStorage #127
Comments
I think this suggestion is fine. |
Oh right, I didn't think about upgrading our Goerli contracts - would need to add some deprecated variables to make up the space. |
Aren't we already planning on redeploying because of other breaking storage layout / semantic changes? This strikes me as the last opportunity to reasonably make the change. |
Will make sure I get a PR open for this today. I wanted to get the integration tests to a good state before I touched the contracts much more |
eigenlayer-middleware/src/StakeRegistryStorage.sol
Lines 30 to 35 in ebe657e
This is the only place in the middleware contracts where we use
[256]
to represent quorum information. I'd like to replace this with a mapping for consistency. I don't think it actually impacts a single thing, but if it this ends up breaking things it's fine to keep as-is.The text was updated successfully, but these errors were encountered: