-
Notifications
You must be signed in to change notification settings - Fork 68
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: use cdk repo for erigon sequencer #193
Conversation
3535f4b
to
0a9a229
Compare
wip wip wip wip almost working fix config config wip Tips and tricks Default to Eldelberry wip fix config Bump prover and erigon
0a9a229
to
3fcbcae
Compare
I'm also running the nightly workflow against your branch to check everything works. edit: looks like the deployment is not working atm and you can also fix lint issues using |
Applied feedback @leovct |
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.
few nits but lgtm! waiting for the deploy
job to pass to approve :)
Quality Gate passedIssues Measures |
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.
lgtm, let's see what the CI says :)
This PR introduce a bunch of changes, running zkevm node is now optional and disabled by default.
The CI is configured for testing both zkevm-node and cdk-erigon + cdk-node while there's still production deployments of the zkevm-node.
This PR goes after 0xPolygon/cdk#4