You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
via with_blockchain_db::<T> and then keep the type T in a phantomType for example.
so we can just call let builder_ctx = ctx.builder_ctx()?
and later by attaching the adapter via fn with_components<N: FullNodeComponents> to the launchcontext with an additional attachment type like WithMeteredProvider
Describe the feature
It would be nice to have access to the
FullNodeComponents
andFullNodeTypes
traits, so we can also move this into the context:reth/crates/node/builder/src/launch/mod.rs
Lines 132 to 140 in 5293a2f
this could be done with a new state type, similar to
reth/crates/node/builder/src/launch/common.rs
Line 685 in 5293a2f
but with access to the
FullNodeTypes
trait, either by providing this herereth/crates/node/builder/src/launch/mod.rs
Line 130 in 5293a2f
via
with_blockchain_db::<T>
and then keep the type T in a phantomType for example.so we can just call
let builder_ctx = ctx.builder_ctx()?
and later by attaching the adapter via
fn with_components<N: FullNodeComponents>
to the launchcontext with an additional attachment type likeWithMeteredProvider
reth/crates/node/builder/src/launch/mod.rs
Lines 168 to 172 in 5293a2f
This way, we can push a ton of setup code currently in
fn launch
into the context.FYI @fgimenez in case you find some time
Additional context
No response
The text was updated successfully, but these errors were encountered: