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

reth-revm-inspectors: Move inspectors out of reth and into their own repo #5956

Closed
Evalir opened this issue Jan 5, 2024 · 3 comments
Closed

reth-revm-inspectors: Move inspectors out of reth and into their own repo #5956

Evalir opened this issue Jan 5, 2024 · 3 comments
Labels
C-debt Refactor of code section that is hard to understand or maintain C-enhancement New feature or request

Comments

@Evalir
Copy link
Collaborator

Evalir commented Jan 5, 2024

Describe the feature

Foundry now depends on reth-revm-inspectors for tracing with foundry-rs/foundry#6219. To remove some burden on foundry having to depend on reth, it would be ideal to extract the inspectors to their own repo, namely https://github.com/paradigmxyz/evm-inspectors. Doing this completes a milestone in foundry-rs/foundry#5216.

Additional context

No response
@Evalir Evalir added C-enhancement New feature or request S-needs-triage This issue needs to be labelled C-debt Refactor of code section that is hard to understand or maintain labels Jan 5, 2024
@Evalir Evalir removed the S-needs-triage This issue needs to be labelled label Jan 5, 2024
@Evalir Evalir mentioned this issue Jan 5, 2024
Closed
43 tasks
@supernovahs
Copy link
Contributor

I can take this

@DaniPopes
Copy link
Member

Hi @supernovahs we're taking this over since it's blocking for further work: https://github.com/paradigmxyz/evm-inspectors

@mattsse
Copy link
Collaborator

mattsse commented Jan 10, 2024

completed

@mattsse mattsse closed this as completed Jan 10, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Reth Tracker Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-debt Refactor of code section that is hard to understand or maintain C-enhancement New feature or request
Projects
Reth Tracker
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mattsse @Evalir @DaniPopes @supernovahs