Add and update types for Capella#60
Conversation
|
@ralexstokes @0xGabi would you mind reviewing this please? I've addressed all comments from the previous PR. This PR includes changes for Capella only. |
|
We should merge this soon :-) |
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
ralexstokes
left a comment
ralexstokes
left a comment
There was a problem hiding this comment.
nice updates! I left some more comments but this is very close to merge
ralexstokes
left a comment
ralexstokes
left a comment
There was a problem hiding this comment.
nice updates! I think the only thing left is to rollback the discriminator change as I think it is unnecessary given the linter update
This reverts commit 702791c.
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
ralexstokes
left a comment
ralexstokes
left a comment
There was a problem hiding this comment.
spectacular! thanks for all of your work here :)
Thank you and everyone else for the review! |
|
For reference, here's the mev-boost issue to track Capella upgrades and further docs: flashbots/mev-boost#451 |
|
btw, thanks again @jimmygchen - awesome work 🙏 |
5 participants
Builder spec changes for Capella, separated from PR #58.
Description
ExecutionPayload&BeaconBlockBodytypes to align with Capella.