From f15f3672b876a7a8f3696cc15eb31181d3f13a23 Mon Sep 17 00:00:00 2001 From: Maurelian Date: Fri, 13 Sep 2024 13:26:28 -0400 Subject: [PATCH] feat: Address feedback and remove comments --- packages/contracts-bedrock/scripts/DeployAuthSystem.s.sol | 7 ------- 1 file changed, 7 deletions(-) diff --git a/packages/contracts-bedrock/scripts/DeployAuthSystem.s.sol b/packages/contracts-bedrock/scripts/DeployAuthSystem.s.sol index 50634679a079f..13ac34ea48c8e 100644 --- a/packages/contracts-bedrock/scripts/DeployAuthSystem.s.sol +++ b/packages/contracts-bedrock/scripts/DeployAuthSystem.s.sol @@ -56,23 +56,16 @@ contract DeployAuthSystemInput is CommonBase { contract DeployAuthSystemOutput is CommonBase { Safe internal _safe; - // This method lets each field be set individually. The selector of an output's getter method - // is used to determine which field to set. function set(bytes4 sel, address _address) public { if (sel == this.safe.selector) _safe = Safe(payable(_address)); else revert("DeployAuthSystemOutput: unknown selector"); } - // Save the output to a TOML file. - // We fetch the output values using external calls to the getters to verify that all outputs are - // set correctly before writing them to the file. function writeOutputFile(string memory _outfile) public { string memory out = vm.serializeAddress("outfile", "safe", address(this.safe())); vm.writeToml(out, _outfile); } - // This function can be called to ensure all outputs are correct. Similar to `writeOutputFile`, - // it fetches the output values using external calls to the getter methods for safety. function checkOutput() public view { address[] memory addrs = Solarray.addresses(address(this.safe())); DeployUtils.assertValidContractAddresses(addrs);