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

Remove dep of revm-interpreter from revm-handler-interface and revm-context #1893

Open
rakita opened this issue Dec 5, 2024 · 0 comments
Open
Labels
good first issue Good for newcomers

Comments

@rakita
Copy link
Member

rakita commented Dec 5, 2024

from this dep graph: #1865 we see this dependency that should be broken.

Both revm-handler-interface and revm-context** depends on revm-interpreter and that should not be the case. Only dependency on revm-interpreter should be from revm-handler as it has execution implementation.

First step is to investigate and than we can decide what to do next.

@rakita rakita added the good first issue Good for newcomers label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant