-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
chore: bump revm #4884
chore: bump revm #4884
Conversation
adds blob fee precision update and fixes builder env bug
Codecov Report
... and 13 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
lgtm
@@ -96,8 +96,7 @@ unknown-registry = "warn" | |||
unknown-git = "deny" | |||
allow-git = [ | |||
# TODO: remove, see ./Cargo.toml | |||
"https://github.com/Evalir/revm", | |||
# "https://github.com/bluealloy/revm", | |||
"https://github.com/bluealloy/revm", |
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.
"https://github.com/bluealloy/revm", |
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.
I think we can keep this, I think we'll have to use via git again eventually
Co-authored-by: rakita <rakita@users.noreply.github.com>
This updates the revm version, which now has alloy types. A bug is also fixed in the payload builder where we were initializing the
BlobExcessGasAndPrice
with the next block's blob fee rather than the next block's excess blob gas.