-
Notifications
You must be signed in to change notification settings - Fork 512
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.
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
Conversation
mattsse
left a comment
There was a problem hiding this 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?
- unskip borsh on unblocked fields
Okay, ping me when you'd like the follow-up with alloy-eip* structs 👍 |
mattsse
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very nice
|
@mablr could you pls rebase and fix the failing CI before merging? thanks! |
|
@grandizzy waiting for next msrv bump before merging this, I messed up msrv on alloy-accesslist and 7702 |
Motivation
Close #2803
Solution
Add Borsh for blocks, receipts, and transactions.
Tracking
TxKindcore#1022PR Checklist