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

Port gas reports #775

Closed
onbjerg opened this issue Feb 19, 2022 · 0 comments · Fixed by #867
Closed

Port gas reports #775

onbjerg opened this issue Feb 19, 2022 · 0 comments · Fixed by #867
Assignees
@onbjerg
Labels
A-gas-snapshots Area: gas snapshotting/reporting C-forge Command: forge Cmd-forge-test Command: forge test T-feature Type: feature

Comments

@onbjerg
Copy link
Collaborator

onbjerg commented Feb 19, 2022

Port gas reports to REVM. I think the ideal case is that we move the gas reporting logic to the CLI instead where we piggyback off of the test traces
@onbjerg onbjerg self-assigned this Feb 19, 2022
@onbjerg onbjerg added Cmd-forge-test Command: forge test C-forge Command: forge A-gas-snapshots Area: gas snapshotting/reporting M-revm T-feature Type: feature labels Feb 19, 2022
@onbjerg onbjerg moved this from Todo to In Progress in Replace Sputnik EVM with Revm Mar 8, 2022
@onbjerg onbjerg mentioned this issue Mar 11, 2022
Merged
@onbjerg onbjerg linked a pull request Mar 11, 2022 that will close this issue
REVM tracing and gas reports #867
Merged
@onbjerg onbjerg mentioned this issue Mar 12, 2022
Merged
18 tasks
@onbjerg onbjerg closed this as completed Mar 13, 2022
Repository owner moved this from In Progress to Done in Replace Sputnik EVM with Revm Mar 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@onbjerg onbjerg

Labels
A-gas-snapshots Area: gas snapshotting/reporting C-forge Command: forge Cmd-forge-test Command: forge test T-feature Type: feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

REVM tracing and gas reports foundry-rs/foundry
1 participant
@onbjerg