Skip to content
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

feat(evm-optimism): add OpBlockAccessListExecutor #10895

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

Rjected
Copy link
Member

@Rjected Rjected commented Sep 13, 2024

Adds the optimism variant of the BlockAccessListExecutor

@Rjected Rjected added C-enhancement New feature or request A-execution Related to the Execution and EVM A-op-reth Related to Optimism and op-reth labels Sep 13, 2024
Copy link
Member

@emhane emhane left a comment

Choose a reason for hiding this comment

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

nice! is there test coverage for this?

@Rjected
Copy link
Member Author

Rjected commented Sep 13, 2024

nice! is there test coverage for this?

There is not

@0x00101010
Copy link
Contributor

@Rjected thanks for implementing this, can we merge this? I'll merge these changes into my branch to make debug_execution_witness work

@Rjected Rjected added this pull request to the merge queue Sep 13, 2024
Merged via the queue into main with commit bb18942 Sep 13, 2024
35 checks passed
@Rjected Rjected deleted the dan/add-op-access-list-executor branch September 13, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-execution Related to the Execution and EVM A-op-reth Related to Optimism and op-reth C-enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants