chore: use named return

contracts/gas-estimation/InterchainGasEstimation.sol

@@ -217,7 +217,7 @@ abstract contract InterchainGasEstimation is IInterchainGasEstimation {
 
         uint256 txSize = _l1TxSize(payload) + overhead + (4 * 16);
 
-        return (l1GasInfo.relativeGasPrice * txSize * gasInfo.l1FeeScalar) / precision;
+        l1DataFee = (l1GasInfo.relativeGasPrice * txSize * gasInfo.l1FeeScalar) / precision;
     }
 
     /**

contracts/gateway/AxelarAmplifierGateway.sol

@@ -113,7 +113,7 @@ contract AxelarAmplifierGateway is BaseAmplifierGateway, BaseWeightedMultisig, U
      * @return isLatestSigners True if provided signers are the current ones
      */
     function validateProof(bytes32 dataHash, Proof calldata proof) external view returns (bool isLatestSigners) {
-        return _validateProof(dataHash, proof);
+        isLatestSigners = _validateProof(dataHash, proof);
     }
 
     /**

contracts/gateway/BaseAmplifierGateway.sol

@@ -170,6 +170,7 @@ abstract contract BaseAmplifierGateway is IBaseAmplifierGateway {
 
     /**
      * @dev For backwards compatibility with `validateContractCall`, `commandId` is used here instead of `messageId`.
+     * @return valid True if message is valid
      */
     function _validateMessage(
         bytes32 commandId,
@@ -224,6 +225,7 @@ abstract contract BaseAmplifierGateway is IBaseAmplifierGateway {
 
     /**
      * @dev For backwards compatibility with `validateContractCall`, `commandId` is used here instead of `messageId`.
+     * @return bytes32 the message hash
      */
     function _messageHash(
         bytes32 commandId,

contracts/governance/BaseMultisig.sol

@@ -53,7 +53,7 @@ contract BaseMultisig is IBaseMultisig {
 
     /**
      * @notice Returns the current signer threshold
-     * @return uint The signer threshold
+     * @return uint256 The signer threshold
      */
     function signerThreshold() external view override returns (uint256) {
         return signers.threshold;

contracts/governance/InterchainGovernance.sol

@@ -174,7 +174,8 @@ contract InterchainGovernance is AxelarGMPExecutable, TimeLock, Caller, IInterch
     }
 
     /**
-     * @dev Get proposal hash using the target, callData, and nativeValue
+     * @notice Get proposal hash using the target, callData, and nativeValue
+     * @return The proposal hash
      */
     function _getProposalHash(
         address target,

contracts/governance/InterchainMultisig.sol

@@ -111,7 +111,7 @@ contract InterchainMultisig is Caller, BaseWeightedMultisig, IInterchainMultisig
      * @return isLatestSigners True if provided signers are the current ones
      */
     function validateProof(bytes32 dataHash, Proof calldata proof) external view returns (bool isLatestSigners) {
-        return _validateProof(dataHash, proof);
+        isLatestSigners = _validateProof(dataHash, proof);
     }
 
     /**

contracts/utils/RolesBase.sol

@@ -57,15 +57,11 @@ contract RolesBase is IRolesBase {
      * @param roles The roles to convert
      * @return accountRoles The roles in uint256 format
      */
-    function _toAccountRoles(uint8[] memory roles) internal pure returns (uint256) {
+    function _toAccountRoles(uint8[] memory roles) internal pure returns (uint256 accountRoles) {
         uint256 length = roles.length;
-        uint256 accountRoles;
-
         for (uint256 i = 0; i < length; ++i) {
             accountRoles |= (1 << roles[i]);
         }
-
-        return accountRoles;
     }
 
     /**
@@ -74,7 +70,7 @@ contract RolesBase is IRolesBase {
      * @return key The key of the roles mapping
      */
     function _rolesKey(address account) internal view virtual returns (bytes32 key) {
-        return keccak256(abi.encodePacked(ROLES_PREFIX, account));
+        key = keccak256(abi.encodePacked(ROLES_PREFIX, account));
     }
 
     /**
@@ -108,7 +104,7 @@ contract RolesBase is IRolesBase {
      * @return key The key of the proposed roles mapping
      */
     function _proposalKey(address fromAccount, address toAccount) internal view virtual returns (bytes32 key) {
-        return keccak256(abi.encodePacked(PROPOSE_ROLES_PREFIX, fromAccount, toAccount));
+        key = keccak256(abi.encodePacked(PROPOSE_ROLES_PREFIX, fromAccount, toAccount));
     }
 
     /**

contracts/utils/TimeLock.sol

@@ -25,7 +25,7 @@ contract TimeLock is ITimeLock {
     /**
      * @notice Returns the timestamp after which the TimeLock may be executed.
      * @param hash The hash of the timelock
-     * @return uint The timestamp after which the timelock with the given hash can be executed
+     * @return uint256 The timestamp after which the timelock with the given hash can be executed
      */
     function getTimeLock(bytes32 hash) external view override returns (uint256) {
         return _getTimeLockEta(hash);

test/governance/AxelarServiceGovernance.js

@@ -411,7 +411,7 @@ describe('AxelarServiceGovernance', () => {
         const bytecodeHash = keccak256(bytecode);
 
         const expected = {
-            london: '0x87891d8e17e62bddae5afa47c6231b236207b7c9cfff0810bc62b226a3765600',
+            london: '0xe11e4b996bf7930c82d83ad8fd28fd34759e7a83b65e5f2644ab8f35295ff7f4',
         }[getEVMVersion()];
 
         expect(bytecodeHash).to.be.equal(expected);

test/governance/InterchainGovernance.js

@@ -380,7 +380,7 @@ describe('InterchainGovernance', () => {
         const bytecodeHash = keccak256(bytecode);
 
         const expected = {
-            london: '0x034b9b57bed553b7c9cfa5e4a14304776b65d8a3caefc87df9339203b04df56e',
+            london: '0xf5821bb7eb3b38c92c9974bbbe9406f966c0fff73a6af31363387abe713345eb',
         }[getEVMVersion()];
 
         expect(bytecodeHash).to.be.equal(expected);