feat: remove unused _includeDump arg (#13099)

* feat: remove unused onlyTestnetOrDevnet modifier * feat: remove unused _includeDump arg
ethereum-optimism · Dec 19, 2024 · 222a7b8 · 222a7b8
1 parent 8942e4b
commit 222a7b8
Showing 1 changed file with 1 addition and 12 deletions.
diff --git a/packages/contracts-bedrock/scripts/deploy/Deploy.s.sol b/packages/contracts-bedrock/scripts/deploy/Deploy.s.sol
@@ -167,14 +167,7 @@ contract Deploy is Deployer {
     /// @notice Deploy a new OP Chain using an existing SuperchainConfig and ProtocolVersions
     /// @param _superchainConfigProxy Address of the existing SuperchainConfig proxy
     /// @param _protocolVersionsProxy Address of the existing ProtocolVersions proxy
-    /// @param _includeDump Whether to include a state dump after deployment
-    function runWithSuperchain(
-        address payable _superchainConfigProxy,
-        address payable _protocolVersionsProxy,
-        bool _includeDump
-    )
-        public
-    {
+    function runWithSuperchain(address payable _superchainConfigProxy, address payable _protocolVersionsProxy) public {
         require(_superchainConfigProxy != address(0), "Deploy: must specify address for superchain config proxy");
         require(_protocolVersionsProxy != address(0), "Deploy: must specify address for protocol versions proxy");
 
@@ -191,10 +184,6 @@ contract Deploy is Deployer {
         save("ProtocolVersionsProxy", _protocolVersionsProxy);
 
         _run(false);
-
-        if (_includeDump) {
-            vm.dumpState(Config.stateDumpPath(""));
-        }
     }
 
     function runWithStateDump() public {