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

docs(book): start documenting ExExes #8779

Merged
merged 16 commits into from
Jun 13, 2024
Merged

docs(book): start documenting ExExes #8779

merged 16 commits into from
Jun 13, 2024

Conversation

shekhirin
Copy link
Collaborator

@shekhirin shekhirin commented Jun 12, 2024

The easiest way to review this is to run mdbook serve locally on this PR and explore the book in your browser.

@shekhirin shekhirin added C-docs An addition or correction to our documentation A-exex Execution Extensions labels Jun 12, 2024
@shekhirin shekhirin requested review from onbjerg and mattsse June 12, 2024 17:21
@shekhirin shekhirin marked this pull request as ready for review June 12, 2024 17:21
@shekhirin shekhirin requested a review from gakonst as a code owner June 12, 2024 17:21
Copy link
Member

@Rjected Rjected left a comment

Choose a reason for hiding this comment

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

looks pretty good! mostly minor comments

book/developers/exex/exex.md Outdated Show resolved Hide resolved
book/developers/exex/exex.md Outdated Show resolved Hide resolved
book/developers/exex/hello-world.md Outdated Show resolved Hide resolved
book/developers/exex/how-it-works.md Outdated Show resolved Hide resolved
Copy link
Member

@onbjerg onbjerg left a comment

Choose a reason for hiding this comment

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

some minor things, looks great!

book/developers/exex/exex.md Outdated Show resolved Hide resolved
book/developers/exex/exex.md Outdated Show resolved Hide resolved
book/developers/exex/exex.md Outdated Show resolved Hide resolved
book/developers/exex/hello-world.md Outdated Show resolved Hide resolved
book/developers/exex/hello-world.md Outdated Show resolved Hide resolved
book/developers/exex/hello-world.md Outdated Show resolved Hide resolved
book/developers/exex/hello-world.md Outdated Show resolved Hide resolved
shekhirin and others added 8 commits June 13, 2024 13:32
Co-authored-by: Dan Cline <6798349+Rjected@users.noreply.github.com>
Co-authored-by: Dan Cline <6798349+Rjected@users.noreply.github.com>
Co-authored-by: Oliver <onbjerg@users.noreply.github.com>
Co-authored-by: Oliver <onbjerg@users.noreply.github.com>
Co-authored-by: Oliver <onbjerg@users.noreply.github.com>
Co-authored-by: Oliver <onbjerg@users.noreply.github.com>
Co-authored-by: Oliver <onbjerg@users.noreply.github.com>
@shekhirin shekhirin enabled auto-merge June 13, 2024 13:16
Comment on lines +73 to +83
fn main() -> eyre::Result<()> {
reth::cli::Cli::parse_args().run(|builder, _| async move {
let handle = builder
.node(EthereumNode::default())
.install_exex("my-exex", |ctx| async move { Ok(my_exex(ctx)) })
.launch()
.await?;

handle.wait_for_node_exit().await
})
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

these snippets will be hard to maintain
but should be okay

Copy link
Collaborator Author

@shekhirin shekhirin Jun 13, 2024

Choose a reason for hiding this comment

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

unfortunately, there seems to be no way to compile snippets that have any dependencies :/

@shekhirin shekhirin added this pull request to the merge queue Jun 13, 2024
Merged via the queue into main with commit f9493fd Jun 13, 2024
29 checks passed
@shekhirin shekhirin deleted the alexey/exex-book branch June 13, 2024 13:41
emhane pushed a commit that referenced this pull request Jun 13, 2024
Co-authored-by: Dan Cline <6798349+Rjected@users.noreply.github.com>
Co-authored-by: Oliver <onbjerg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-exex Execution Extensions C-docs An addition or correction to our documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants