Skip to content

feat(consensus,eips,genesis): add Borsh support #2946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

feat(consensus,eips,genesis): add Borsh support #2946

wants to merge 7 commits into from

Conversation

mablr
Copy link
Contributor

@mablr mablr commented Sep 28, 2025 •
edited
Loading

Motivation

Close #2803

Solution

Add Borsh for blocks, receipts, and transactions.

Tracking

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

This was referenced Sep 29, 2025
Closed
Closed
mablr
mablr commented Sep 29, 2025
View reviewed changes
mattsse
mattsse requested changes Oct 15, 2025
View reviewed changes
Copy link
Member

@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.

this should be unblocked now wiht alloy-core 1.4.1

let's limit this to just the consensus (tx,receipt) types first?

@github-project-automation github-project-automation bot moved this to In Progress in Alloy Oct 15, 2025
@mablr mablr marked this pull request as ready for review October 16, 2025 08:17
@mablr mablr requested a review from mattsse October 16, 2025 08:17
@mablr
Copy link
Contributor Author

mablr commented Oct 16, 2025

let's limit this to just the consensus (tx,receipt) types first?

Okay, ping me when you'd like the follow-up with alloy-eip* structs 👍

mattsse
mattsse approved these changes Oct 17, 2025
View reviewed changes
Copy link
Member

@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.

very nice

@github-project-automation github-project-automation bot moved this from In Progress to Reviewed in Alloy Oct 17, 2025
@mattsse mattsse mentioned this pull request Oct 17, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattsse mattsse mattsse approved these changes

@DaniPopes DaniPopes Awaiting requested review from DaniPopes DaniPopes is a code owner

@klkvr klkvr Awaiting requested review from klkvr klkvr is a code owner

@onbjerg onbjerg Awaiting requested review from onbjerg onbjerg is a code owner

@zerosnacks zerosnacks Awaiting requested review from zerosnacks zerosnacks is a code owner

@grandizzy grandizzy Awaiting requested review from grandizzy grandizzy is a code owner

Assignees

No one assigned

Labels

None yet

Projects

Alloy
Status: Reviewed

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature] Add borsh support for block header and tx types

2 participants

@mablr @mattsse