-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add support for sepolia #57
Add support for sepolia #57
Conversation
46b5afa
to
7c97db5
Compare
Oki, for some reason, if i copy the specs it just fail computing the state root!? I have to let only 1 timestamp and the rest to 0s, don't understand why yet TT ( i fix it in this commit) |
Thanks for this PR @invocamanman, can you please solve the conflicts? |
784ca2d
to
bf51cef
Compare
done ^^ @leruaa |
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.
Looking good, only a small nit.
I also launched CI.
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.
ty!
I fixed the remaining CI failures.
Let me know if i have to change something else^^
It works ( on my computer :P)
I bump the
sp1-sdk
to 2.0.0 version since the proof verification said:Sepolia spec took if from here: https://github.com/paradigmxyz/reth/blob/b4d5adeeb1a0a3dce0c6e49b102910e4e072f7a0/crates/chainspec/src/spec.rs#L58