Skip to content

fix(payload): Ethereum Payload Builder Zero Type#5313

Merged
mattsse merged 9 commits intoparadigmxyz:mainfrom
refcell:refcell/eth-payload-builder
Nov 6, 2023
Merged

fix(payload): Ethereum Payload Builder Zero Type#5313
mattsse merged 9 commits intoparadigmxyz:mainfrom
refcell:refcell/eth-payload-builder

Conversation

@refcell
Copy link
Contributor

@refcell refcell commented Nov 5, 2023

Description

Defines an EthereumPayloadBuilder type as the empty tuple type following @gakonst's comment in #4377 (comment)

@refcell refcell changed the title fix(payload): Ethereum Payload Builder fix(payload): Ethereum Payload Builder [RFC] Nov 5, 2023
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the default usage also needs to be updated:

pub struct BasicPayloadJobGenerator<Client, Pool, Tasks, Builder = ()> {

@refcell refcell requested a review from mattsse November 5, 2023 19:34
@refcell refcell changed the title fix(payload): Ethereum Payload Builder [RFC] fix(payload): Ethereum Payload Builder Nov 5, 2023
@refcell refcell changed the title fix(payload): Ethereum Payload Builder fix(payload): Ethereum Payload Builder Zero Type Nov 5, 2023
@mattsse mattsse enabled auto-merge November 6, 2023 12:44
@mattsse mattsse added the A-block-building Related to block building label Nov 6, 2023
@mattsse mattsse added this pull request to the merge queue Nov 6, 2023
Merged via the queue into paradigmxyz:main with commit 22dfe96 Nov 6, 2023
mattsse added a commit that referenced this pull request Nov 8, 2023
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-block-building Related to block building

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants