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

new tracers #2663

Merged
merged 9 commits into from
Nov 23, 2023
Merged

new tracers #2663

merged 9 commits into from
Nov 23, 2023

Conversation

tclemos
Copy link
Contributor

@tclemos tclemos commented Oct 14, 2023

Closes #2662.
Closes #2744.

What does this PR do?

adapt code to work with the new full tracer

@tclemos tclemos added this to the v0.5.0 milestone Oct 14, 2023
@tclemos tclemos self-assigned this Oct 14, 2023
@cla-bot cla-bot bot added the cla-signed label Oct 14, 2023
@tclemos tclemos marked this pull request as ready for review November 8, 2023 21:53
@tclemos tclemos requested a review from ToniRamirezM as a code owner November 8, 2023 21:53
@tclemos tclemos modified the milestones: v0.6.0, v0.5.0 Nov 21, 2023
@tclemos
Copy link
Contributor Author

tclemos commented Nov 23, 2023

The executor version is still pending to be generated, but the changes for Etrog are requiring the changes in this PR to make the proto files compatible with the State.

@tclemos tclemos merged commit d7630ce into develop Nov 23, 2023
17 checks passed
@tclemos tclemos deleted the feature/replace-default-tracer-by-geth branch November 23, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong gas_used in internal transaction refactoring to be compatible with full trace
2 participants