-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: add cli ext event hooks example #4935
Conversation
8a776fc
to
800c215
Compare
800c215
to
5f6fd14
Compare
Codecov Report
... and 7 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are examples of things one would want to do with on_components_initialized
and on_node_started
?
[dependencies] | ||
reth.workspace = true | ||
clap.workspace = true | ||
eyre = "0.6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so clean!
add an example for how to use the new reth config hooks with no additional cli arguments
This adds a
NoArgsCliExt
that makes it easier to configure a NodeConfig that doesn't take additional cli args