Skip to content

[wip] feat: integrate new revm#14115

Merged
klkvr merged 15 commits intoklkvr/alloy-evmfrom
klkvr/new-revm
Feb 17, 2025
Merged

[wip] feat: integrate new revm#14115
klkvr merged 15 commits intoklkvr/alloy-evmfrom
klkvr/new-revm

Conversation

@klkvr
Copy link
Member

@klkvr klkvr commented Jan 30, 2025

Based on #14021

Updates revm to latest

@emhane emhane added A-execution Related to the Execution and EVM A-dependencies Pull requests or issues that are about dependencies labels Feb 4, 2025
@klkvr klkvr force-pushed the klkvr/alloy-evm branch 4 times, most recently from e5049d7 to caef4ba Compare February 7, 2025 00:49
@klkvr klkvr force-pushed the klkvr/new-revm branch 2 times, most recently from a1c7307 to 4619a86 Compare February 12, 2025 17:57
Copy link
Collaborator

@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.

let's do this

Comment on lines +753 to 754
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we should reconsider this now and remove the fill function and instead do -> TxEnv

Copy link
Collaborator

Choose a reason for hiding this comment

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

hmm, why does this exist?

Comment on lines +98 to +102
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is so much worse now imo -.-

klkvr and others added 3 commits February 17, 2025 21:28
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Co-authored-by: Arsenii Kulikov <klkvrr@gmail.com>
@klkvr klkvr marked this pull request as ready for review February 17, 2025 19:35
@klkvr klkvr merged commit 9bf7709 into klkvr/alloy-evm Feb 17, 2025
36 checks passed
@klkvr klkvr deleted the klkvr/new-revm branch February 17, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-dependencies Pull requests or issues that are about dependencies A-execution Related to the Execution and EVM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants