-
Notifications
You must be signed in to change notification settings - Fork 446
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
Withdrawals wei -> gwei #5139
Withdrawals wei -> gwei #5139
Conversation
src/Nethermind/Nethermind.Serialization.Rlp/WithdrawalDecoder.cs
Outdated
Show resolved
Hide resolved
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.
GWei
should be changed to Gwei
.
src/Nethermind/Nethermind.Serialization.Rlp/WithdrawalDecoder.cs
Outdated
Show resolved
Hide resolved
src/Nethermind/Nethermind.Serialization.Rlp/WithdrawalDecoder.cs
Outdated
Show resolved
Hide resolved
@MarekM25 Could you please keep following our PR template? |
sure, sorry about it :) |
e418ad5
to
1eac7f4
Compare
@MarekM25 I think we can merge it as I implemented ethereum/execution-apis#337 that's based on this PR. Hive tests passing. |
Don't merge for now.
Engine API: ethereum/execution-apis#354
EIP-4895: ethereum/EIPs#6325