Use ProgressiveTransactions type for the IL transactions field - #9747
Conversation
|
A heads up, The same spec PR also changes IL production: |
|
once conflicts are resolved ill merge |
Just resolved the conflicts. Should be good to go 🙏 |
Merge Queue Status
This pull request spent 5 minutes 16 seconds in the queue, including 3 minutes 11 seconds running CI. Waiting for
All conditions
ReasonThe merge conditions cannot be satisfied due to failing checks
HintYou may have to fix your CI before adding the pull request to the queue again. Requeued — the merge queue status continues in this comment ↓. |
|
looks like some clippy issues @conache make sure to run |
|
@mergify requeue |
Merge Queue Status
This pull request spent 29 minutes 40 seconds in the queue, including 28 minutes 18 seconds running CI. Required conditions to merge
|
☑️ This pull request is already queued |
Issue Addressed
Follow-up to #9708 , resolving the outstanding switch of
InclusionList.transactionstoProgressiveTransactions.This was unblocked by the merge of #9450.
Proposed Changes
InclusionList.transactionsfromTransactions<E>toProgressiveTransactions, matching the spec typeEthSpecanymore,InclusionListandSignedInclusionListdrop this type parameter