fix

base · Jan 10, 2025 · 78b43c2 · 78b43c2
1 parent 6316505
commit 78b43c2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/mainnet/2025-01-09-add-security-council/README.md b/mainnet/2025-01-09-add-security-council/README.md
@@ -13,7 +13,7 @@ This script swaps out the Coinbase signer 3-of-6 multisig for a new Coinbase sig
 
 a. Signers TBD
 
-### 2. Add new Security Council address to `.env` file as the `BASE_SECURITY_COUNCIL` variable
+### 2. Add new Security Council address to `.env` file as the `SECURITY_COUNCIL` variable
 
 ### 3. Deploy "Coinbase" signer 2-of-2 multisig with signers:
 

diff --git a/mainnet/2025-01-09-add-security-council/script/AddSecurityCouncil.s.sol b/mainnet/2025-01-09-add-security-council/script/AddSecurityCouncil.s.sol
@@ -11,7 +11,7 @@ import "@base-contracts/script/universal/NestedMultisigBuilder.sol";
 // Gov (CB_new, OP)
 // CB_new (Base_SC, CB_old)
 contract AddSecurityCouncil is NestedMultisigBuilder {
-    address internal _BASE_SECURITY_COUNCIL = vm.envAddress("BASE_SECURITY_COUNCIL");
+    address internal _SECURITY_COUNCIL = vm.envAddress("SECURITY_COUNCIL");
     address internal _COINBASE_SIGNER_NEW = vm.envAddress("COINBASE_SIGNER_NEW");
     address internal _COINBASE_SIGNER_CURRENT = vm.envAddress("COINBASE_SIGNER_CURRENT");
     address internal _GOVERNANCE_MULTISIG = vm.envAddress("GOVERNANCE_MULTISIG");
@@ -26,7 +26,7 @@ contract AddSecurityCouncil is NestedMultisigBuilder {
         require(IGnosisSafe(_GOVERNANCE_MULTISIG).isOwner(_OP_MULTISIG), "OP multisig is not owner of governance multisig");
 
         require(IGnosisSafe(_COINBASE_SIGNER_NEW).getOwners().length == 2, "New CB Signer multisig should have 2 owners");
-        require(IGnosisSafe(_COINBASE_SIGNER_NEW).isOwner(_BASE_SECURITY_COUNCIL), "Base Security Council is not owner of new CB Signer multisig");
+        require(IGnosisSafe(_COINBASE_SIGNER_NEW).isOwner(_SECURITY_COUNCIL), "Base Security Council is not owner of new CB Signer multisig");
         require(IGnosisSafe(_COINBASE_SIGNER_NEW).isOwner(_COINBASE_SIGNER_CURRENT), "Current CB Signer is not owner of new CB Signer multisig");
     }
 
@@ -37,7 +37,7 @@ contract AddSecurityCouncil is NestedMultisigBuilder {
         require(IGnosisSafe(_GOVERNANCE_MULTISIG).isOwner(_COINBASE_SIGNER_NEW), "New CB Signer multisig is not owner of governance multisig");
 
         require(IGnosisSafe(_COINBASE_SIGNER_NEW).getOwners().length == 2, "New CB Signer multisig should have 2 owners");
-        require(IGnosisSafe(_COINBASE_SIGNER_NEW).isOwner(_BASE_SECURITY_COUNCIL), "Base Security Council is not owner of new CB Signer multisig");
+        require(IGnosisSafe(_COINBASE_SIGNER_NEW).isOwner(_SECURITY_COUNCIL), "Base Security Council is not owner of new CB Signer multisig");
         require(IGnosisSafe(_COINBASE_SIGNER_NEW).isOwner(_COINBASE_SIGNER_CURRENT), "Current CB Signer is not owner of new CB Signer multisig");
     }