-
Notifications
You must be signed in to change notification settings - Fork 10
Status L2 Linea test deployment #226
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
base: master
Are you sure you want to change the base?
Conversation
Hmm, maybe we should commit these files so we can re-deploy on Linea if needed? IOW, maybe we should take this out of draft? Thoughts? |
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.
Overall I think this looks good. However, is there a particular reason for wanting to commit the zkey and r1cs? These are both very large files (and could get a lot larger).
There is no specific reason, I wasn't sure the importance to add them into git or not. |
Those files are committed for other networks, like testnet for example. |
Ah yes, I remember we moved them from |
@emizzle, I would assume these files (zkey and r1cs) are similar to the existing network ( I was added them following an example for @AuHau was first, let's ask him for more details - why do we need zkey and r1cs if we have them already on CDN? |
No description provided.