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

feat(rollup): create rollup pipeline from kona_derive #31

Merged
merged 11 commits into from
Aug 26, 2024

Conversation

merklefruit
Copy link
Collaborator

@merklefruit merklefruit commented Aug 25, 2024

Adds an utility to create a kona_derive::DerivationPipeline implementation suitable for the rollup node.
In particular, I tried to keep only the providers as generic, because the actual stage implementations will
most likely be the same for the exex and the standalone mode.

addresses part of #8.

@merklefruit merklefruit force-pushed the nico/feat/new-pipeline branch from 22a2cc0 to 7ee5988 Compare August 25, 2024 18:12
@merklefruit merklefruit requested a review from refcell August 25, 2024 19:37
@merklefruit merklefruit force-pushed the nico/feat/new-pipeline branch from 7ee5988 to 00f9f79 Compare August 25, 2024 20:05
@shekhirin shekhirin added C-enhancement New feature or request A-rollup A-hera Area: OP Stack Rollup node binary and removed A-rollup labels Aug 25, 2024
Copy link
Collaborator

@refcell refcell left a comment

Choose a reason for hiding this comment

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

LGTM

@merklefruit merklefruit requested a review from shekhirin August 26, 2024 11:52
@merklefruit merklefruit force-pushed the nico/feat/new-pipeline branch from 4d99825 to 39ee1cc Compare August 26, 2024 17:31
@merklefruit merklefruit merged commit 6f9d2a7 into ithacaxyz:main Aug 26, 2024
11 checks passed
@merklefruit merklefruit deleted the nico/feat/new-pipeline branch August 26, 2024 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-hera Area: OP Stack Rollup node binary C-enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants