Update op-deployer and foundry scripts for Kurtosis devnet on celo-rebase-13#396
Update op-deployer and foundry scripts for Kurtosis devnet on celo-rebase-13#396Kourin1996 wants to merge 4 commits intopahor/newTestnetDeployfrom
Conversation
|
Overall changes look good One thing that I'm missing is CeloSuperchainConfig (I'm not sure how much aligned with mainnet do you want to be). We currently chose different approach with introducing CustomGasToken and CeloSuperchainConfig for Celo Sepolia Testnet - https://github.com/celo-org/optimism/commits/pahor/newTestnetDeploy (you can check few last commits that are specifically addressing this) |
0058e4a to
f8aeb37
Compare
|
@pahor167 Hi, I adjusted my code for new version of op-deployer and switched the target branch to pahor/newTestnetDeploy. My change should not affect to normal deployment without new overrides. I would be appreciate if you could check it at your convenience. Thank you! |
There was a problem hiding this comment.
Hard to say something specific -> but I would compare this file with the one from martinvol/completeMigrations: https://github.com/celo-org/optimism/blob/martinvol/completeMigrations/packages/contracts-bedrock/scripts/L2Genesis.s.sol
And discuss potential differences with @martinvol -> since this one is the base for changes in Migration.s.sol
This reverts commit d40eb9c.
This PR adds optional intents to deploy Celo custom gas token on L1 and CELO-specific L2 contracts, enabling Kurtosis devnet to run on celo-rebase-13
These flags are enabled through the intent shown in the following code:
https://github.com/celo-org/optimism-package/blob/3d50e7f2503b26f782c134aaa1c649088a83511a/src/contracts/contract_deployer.star#L201-L206
Address https://github.com/celo-org/celo-blockchain-planning/issues/893, https://github.com/celo-org/celo-blockchain-planning/issues/894