Skip to content

Conversation

@mablr
Copy link
Contributor

@mablr mablr commented Sep 28, 2025

Motivation

Close #2803

Solution

Add Borsh for blocks, receipts, and transactions.

Tracking

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking 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 👍

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
@grandizzy
Copy link
Contributor

@mablr could you pls rebase and fix the failing CI before merging? thanks!

@mattsse
Copy link
Member

mattsse commented Oct 28, 2025

@grandizzy waiting for next msrv bump before merging this, I messed up msrv on alloy-accesslist and 7702

@mattsse mattsse merged commit 808b88d into alloy-rs:main Nov 5, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

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

3 participants