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

Specific Configs to GethDebugTracerConfig + generic config build method for GethDebugTracingOptions #686

Merged
merged 2 commits into from
May 22, 2024

Conversation

halo3mic
Copy link
Contributor

@halo3mic halo3mic commented May 5, 2024

Motivation

  1. Simpler conversion from specific tracer config to GethDebugTracerConfig
  2. Single config build method on GethDebugTracingOptions that covers all configs

Solution

  1. Impl From for CallConfig, PreStateConfig, MuxConfig
  2. Introduce generic build method for trace configs on GethDebugTracingOptions

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

…c build method for config on GethDebugTracingOptions
@DaniPopes
Copy link
Member

cc @mattsse

@yash-atreya
Copy link
Member

@halo3mic are you still on this? Else I can take this over, we'd like to wrap it up

@halo3mic
Copy link
Contributor Author

halo3mic commented May 22, 2024

@halo3mic are you still on this? Else I can take this over, we'd like to wrap it up

Hey, @yash-atreya yea I did the changes that were requested in the latest commit. Should've replied saying changes were made, sorry!

@mattsse mattsse merged commit 1ee5ace into alloy-rs:main May 22, 2024
24 checks passed
ben186 pushed a commit to ben186/alloy that referenced this pull request Jul 27, 2024
…od for GethDebugTracingOptions (alloy-rs#686)

* add conversion from specific config to GethDebugTracerConfig + generic build method for config on GethDebugTracingOptions

* add back call_config & prestate_config helper functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants