-
Notifications
You must be signed in to change notification settings - Fork 70
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
wip: predpeloy bytecode validation (approach 1) #475
Conversation
validation/foundry-config.patch
Outdated
- "@cwia/=node_modules/clones-with-immutable-args/src", | ||
+ "@cwia/=node_modules/clones-with-immutable-args", |
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.
This patch file is needed to compile contracts in the monorepo at the specified commit.
validation/genesis-predeploy_test.go
Outdated
|
||
chainId := uint64(34443) // Mode mainnet | ||
|
||
monorepoCommit := "d80c145e0acf23a49c6a6588524f57e32e33b91c" |
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.
This commit was provided by the security team as the commit used by conduit to generate the genesis for Mode.
needs a fix
* add source command back to shell script for local testing * refactor predeploy test file with more functions * justfile: fix regex to only run single CHAIN_ID, not supersets * validation: optimize predeploy amount of code run per chain * refactor predeploy test to remove some lines of code --------- Co-authored-by: George Knee <[email protected]>
In favour of #489 , we won't delete the branch yet though. |
Next steps:
Towards #372