Skip to content

feat: made TxDeposit's mint field non-optional#514

Merged
mattsse merged 6 commits intoalloy-rs:mainfrom
Rimeeeeee:mint-nonoptional
May 23, 2025
Merged

feat: made TxDeposit's mint field non-optional#514
mattsse merged 6 commits intoalloy-rs:mainfrom
Rimeeeeee:mint-nonoptional

Conversation

@Rimeeeeee
Copy link
Copy Markdown
Contributor

fixes #513

Copy link
Copy Markdown
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.

great

maybe let's add a few rlp roundtrip tests as well

you can get raw deposit txs from basescan for example

the first tx in a block is awlays a deposit, for example:

https://basescan.org/getRawTx?tx=0xa68157af562dec44406bf9334381cf7e46aab0dd3f6495d99644e02b6fed6f09

Comment thread crates/consensus/src/transaction/deposit.rs Outdated
Comment thread crates/consensus/src/transaction/deposit.rs Outdated
Comment thread crates/consensus/src/transaction/envelope.rs Outdated
Comment thread crates/consensus/src/transaction/deposit.rs Outdated
@github-project-automation github-project-automation Bot moved this to In Review in Project Tracking May 8, 2025
@Rimeeeeee Rimeeeeee requested a review from mattsse May 8, 2025 13:11
Copy link
Copy Markdown
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.

last nit, then this is g2m

will block merge shortly

cc @clabby for vis

Comment thread crates/consensus/src/transaction/deposit.rs Outdated
Copy link
Copy Markdown
Collaborator

@clabby clabby left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for picking this up.

Will defer to @mattsse to merge when he's ready.

@mattsse mattsse added this pull request to the merge queue May 23, 2025
Merged via the queue into alloy-rs:main with commit cfe1e81 May 23, 2025
21 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Project Tracking May 23, 2025
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
fixes #513

---------

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Make TxDeposit's mint field non-optional

3 participants