EIP-7688: Forward Compatible Consensus Data Structures - #9450
Merged
Conversation
6 tasks
Member
|
5436 was also added as part of the alpha spec 12 changes |
eserilev
reviewed
Aug 3, 2026
eserilev
left a comment
Member
There was a problem hiding this comment.
Finished my review, outside of my nits i think this should be g2g. imo I think we should get this into unstable asap as its blocking alpha spec 12 and 13
eserilev
reviewed
Aug 3, 2026
eserilev
reviewed
Aug 3, 2026
|
Some required checks have failed. Could you please take a look @macladson? 🙏 |
Member
Author
|
For the other Claude findings I have addressed 3 and 5, but left 1, 2 and 4 (size-bounds and limits) for a follow-up PR. |
Merge Queue Status
This pull request spent 29 minutes 35 seconds in the queue, including 27 minutes 39 seconds running CI. Required conditions to merge
|
8 tasks
mergify Bot
pushed a commit
that referenced
this pull request
Aug 12, 2026
## Issue Addressed Follow-up to #9708 , resolving the outstanding switch of `InclusionList.transactions` to `ProgressiveTransactions`. This was unblocked by the merge of #9450. ## Proposed Changes - Switch `InclusionList.transactions` from `Transactions<E>` to `ProgressiveTransactions`, matching [the spec type](https://github.com/ethereum/consensus-specs/blob/master/specs/heze/beacon-chain.md#inclusionlist) - Since no field depends on `EthSpec` anymore, `InclusionList` and `SignedInclusionList` drop this type parameter
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Relevant Links
Proposed Changes
Adds the necessary types and components to support EIP-7688, along with the cargo patches to pull in the
progressiveversions of the crates in our SSZ stack.Testing
Additional Info
Based on the original work by @michaelsproul in #8505