-
Notifications
You must be signed in to change notification settings - Fork 79
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
Add mergeForkBlock #614
Add mergeForkBlock #614
Conversation
Codecov Report
@@ Coverage Diff @@
## master #614 +/- ##
==========================================
- Coverage 81.65% 81.64% -0.01%
==========================================
Files 167 167
Lines 13925 13928 +3
==========================================
+ Hits 11370 11372 +2
- Misses 2555 2556 +1
Continue to review full report at Codecov.
|
Silkworm uses uint64 for block numbers. Why do you want to change it to uint256? |
To match Erigon using big int in terminalBlockNumber |
And why Erigon uses such a huge var type ? |
mergeForkBlock
corresponds toFORK_NEXT_VALUE
of EIP-3675.