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.

Sign up for GitHub

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

Jump to bottom

Refactor REVM to make optimism feature flag additive #464

Open
Wodann opened this issue May 22, 2024 · 1 comment
Open

Refactor REVM to make optimism feature flag additive #464

Wodann opened this issue May 22, 2024 · 1 comment
Assignees
@Wodann

Comments

@Wodann
Copy link
Member

Wodann commented May 22, 2024

Currently, the optimism feature flag removes part of the default feature set of REVM. This prevents us from using vanilla Ethereum and Optimism side-by-side.

Definition of Done

  • Ensure that optimism feature flag in REVM is additive
  • Validate that we can use vanilla Ethereum and Optimism features side-by-side in EDR
@Wodann Wodann self-assigned this May 22, 2024
@Wodann Wodann added this to EDR May 22, 2024
@Wodann Wodann assigned fvictorio and unassigned Wodann May 22, 2024
@github-project-automation github-project-automation bot moved this to Unconfirmed in EDR May 22, 2024
@Wodann Wodann assigned Wodann and unassigned fvictorio May 22, 2024
@Wodann Wodann moved this from Unconfirmed to In Progress in EDR May 22, 2024
@Wodann Wodann linked a pull request May 22, 2024 that will close this issue
refactor: allow chain-specific configuration of Evm bluealloy/revm#1378
Closed
@Wodann Wodann moved this from In Progress to Under Review in EDR Jun 4, 2024
@Wodann Wodann mentioned this issue Jun 6, 2024
Closed
@Wodann
Copy link
Member Author

Wodann commented Jun 6, 2024 •
edited
Loading

Follow-up task: #495

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Wodann Wodann

Labels
None yet
Projects
EDR
Status: Under Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: allow chain-specific configuration of Evm NomicFoundation/revm
2 participants
@fvictorio @Wodann