Merge pull request #8456 from ethereum-optimism/ctb/deny-warning

contracts-bedrock: fail on warnings

.circleci/config.yml

@@ -161,6 +161,8 @@ jobs:
           command: forge --version
       - run:
           name: Build monorepo
+          environment:
+            FOUNDRY_PROFILE: ci
           command: pnpm build
       - persist_to_workspace:
           root: "."

packages/contracts-bedrock/foundry.toml

@@ -53,8 +53,9 @@ ignore = ['src/vendor/WETH9.sol']
 #                         PROFILE: CI                          #
 ################################################################
 
-[profile.ci.fuzz]
-runs = 512
+[profile.ci]
+deny_warnings = true
+fuzz = { runs = 512 }
 
 ################################################################
 #                         PROFILE: LITE                        #

packages/contracts-bedrock/scripts/Deploy.s.sol

@@ -812,7 +812,6 @@ contract Deploy is Deployer {
     /// @notice Initialize the L1ERC721Bridge
     function initializeL1ERC721Bridge() public broadcast {
         console.log("Upgrading and initializing L1ERC721Bridge proxy");
-        ProxyAdmin proxyAdmin = ProxyAdmin(mustGetAddress("ProxyAdmin"));
         address l1ERC721BridgeProxy = mustGetAddress("L1ERC721BridgeProxy");
         address l1ERC721Bridge = mustGetAddress("L1ERC721Bridge");
         address superchainConfigProxy = mustGetAddress("SuperchainConfigProxy");