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

fix: update fork config #645

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: update fork config #645

wants to merge 2 commits into from

Conversation

QEDK
Copy link

@QEDK QEDK commented Oct 16, 2023

  • The given fork.4 config is not compatible with fork.5 prover, hence updated
  • Removed $ so commands are copiable

@cla-bot
Copy link

cla-bot bot commented Oct 16, 2023

We require contributors/corporates @QEDK to read our Contributor License Agreement, please check the Individual CLA document/Corporate CLA document

@cla-bot
Copy link

cla-bot bot commented Oct 16, 2023

We require contributors/corporates @QEDK to read our Contributor License Agreement, please check the Individual CLA document/Corporate CLA document

@QEDK
Copy link
Author

QEDK commented Oct 16, 2023

Maybe I'm missing something but I don't see a way to sign the CLA. 🤔

@vtleonardo
Copy link

This looks good to me. I wonder what needs to be reviewed and by whom. I painfully downloaded 70gb in vain following the README with the fork.4 info, to discover that it doesn't work with the latest code. Following the README with this PR did the trick for me. The other repos have info about the fork.5 for a long time and seem that the team is preparing for fork.6 soon. I wonder where we can get the change logs and roadmap for the forks since now this is an important piece of software that Polygon foundation is advertizing to the community (Polygon CDK).

@jastisriradheshyam
Copy link

Is there any documentation to generate these config files for any particular fork ID ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants