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

rpcdaemon: gas cost from EVMONE for debug family API #2325

Merged
merged 7 commits into from
Sep 10, 2024

Conversation

Sixtysixter
Copy link
Contributor

@Sixtysixter Sixtysixter commented Sep 8, 2024

Relevant changes:

  • use customised tracing API from our evmone fork
  • get the internal gas cost from EVMONE for debug_trace* API

@Sixtysixter Sixtysixter force-pushed the feature/gas_cost_from_evmone branch from ec87872 to fab16dc Compare September 10, 2024 06:56
@Sixtysixter Sixtysixter marked this pull request as ready for review September 10, 2024 08:48
@canepat canepat added the maintenance Some maintenance work (fix, refactor, rename, test...) label Sep 10, 2024
Copy link
Member

@canepat canepat left a comment

Choose a reason for hiding this comment

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

Even if the code is not clean enough yet, I'm really keen on merging this PR because it introduces the improved evmone tracing API.
@Sixtysixter as discussed, you will do a follow-up PR to clean up the code

@canepat canepat changed the title rpcdaemon: gas cost from EVMONE for debug_trace family API rpcdaemon: gas cost from EVMONE for debug family API Sep 10, 2024
@canepat canepat merged commit ca8d940 into master Sep 10, 2024
5 checks passed
@canepat canepat deleted the feature/gas_cost_from_evmone branch September 10, 2024 09:13
@canepat canepat mentioned this pull request Sep 10, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Some maintenance work (fix, refactor, rename, test...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants