Skip to content

Add support for Deneb EIP-4844 types (blobs support)#77

Merged
ralexstokes merged 9 commits intoralexstokes:mainfrom
jimmygchen:add-deneb-types
Aug 3, 2023
Merged

Add support for Deneb EIP-4844 types (blobs support)#77
ralexstokes merged 9 commits intoralexstokes:mainfrom
jimmygchen:add-deneb-types

Conversation

@jimmygchen
Copy link
Copy Markdown
Contributor

@jimmygchen jimmygchen commented Feb 16, 2023

Builder spec: ethereum/builder-specs#61

This PR updates the Deneb EIP-4844 types only, logic for supporting blobs are not implemented.

@jimmygchen jimmygchen marked this pull request as ready for review July 12, 2023 13:48
@jimmygchen
Copy link
Copy Markdown
Contributor Author

Hey @ralexstokes 👋
I think this PR is ready for a round of review when you have some time, thanks!

@jimmygchen jimmygchen changed the title Add support for Deneb types Add support for Deneb EIP-4844 types (blobs support) Jul 12, 2023
Copy link
Copy Markdown
Owner

@ralexstokes ralexstokes left a comment

Choose a reason for hiding this comment

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

thanks for this @jimmygchen !

it may need one more rebase from main but I think it is good enough to merge for now

# Conflicts:
#	Cargo.lock
#	Cargo.toml
@jimmygchen
Copy link
Copy Markdown
Contributor Author

Thanks @ralexstokes I'm seeing a test failure locally, looking into it now

@jimmygchen
Copy link
Copy Markdown
Contributor Author

Turned out it was a local issue, CI is passing locally now!

@ralexstokes ralexstokes merged commit 9d88a23 into ralexstokes:main Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants