• Mark Tyneway's avatar
    contracts-bedrock: delete dead code (#10359) · 40421e86
    Mark Tyneway authored
    * contracts-bedrock: simplify L1 state dump
    
    Removes the need to specify the L1 chainid via CLI.
    Follows similar patterns to L2 genesis generation that
    were done recently.
    
    Follow up PRs can potentially use the same sort of deployer pattern
    as in https://github.com/ethereum-optimism/optimism/pull/10343
    for when doing the L1 genesis dump to remove the need to specify the
    private key.
    
    * contracts-bedrock: explicit deploy config
    
    Remove the concept of implicit deploy config for explicit deploy config.
    This removes confusing implicit behavior as well as makes it much
    more straight forward for deploying multiple superchain targets
    to the same L1. This is a breaking change but the error message
    makes it very obvious and the docs are being updated in a way
    that should include this information.
    
    * contracts-bedrock: fix possible error
    
    * config: cleanup
    
    * deploy-config: fixup
    
    * contracts-bedrock: delete name function
    
    The `name()` function existed due to legacy
    purposes when dealing with hardhat artifacts
    and no longer is necessary. This commit removes
    it in favor of a simpler approach of just using
    the chainid instead of the name. The files that
    are written are not committed into the repo.
    
    * contracts: delete dead code
    
    * lint: fix
    
    * kontrol: fixup
    
    * contracts-bedrock: prevent error
    
    * kontrol: fixup
    
    * kontrol: fix script
    
    * gitignore: update
    
    * devnet: simplify
    
    * Update packages/contracts-bedrock/scripts/Config.sol
    Co-authored-by: default avatarMatt Solomon <matt@mattsolomon.dev>
    
    ---------
    Co-authored-by: default avatarMatt Solomon <matt@mattsolomon.dev>
    40421e86
DeployPeriphery.s.sol 24.4 KB