fix build issues

Author: nlordell

contracts/Safe.sol

@@ -56,14 +56,14 @@ contract Safe is
     string public constant override VERSION = "1.5.0";
 
     /**
-     * @notice The precomputed EIP-712 domain separator hash for Safe typed data hashing and signing.
-     * @dev Precomputed value of: `keccak256("EIP712Domain(uint256 chainId,address verifyingContract)")`.
+     * @dev The precomputed EIP-712 domain separator hash for Safe typed data hashing and signing.
+     *      Precomputed value of: `keccak256("EIP712Domain(uint256 chainId,address verifyingContract)")`.
      */
     bytes32 private constant DOMAIN_SEPARATOR_TYPEHASH = 0x47e79534a245952e8b16893a336b85a3d9ea9fa8c573f3d803afb92a79469218;
 
     /**
-     * @notice The precomputed EIP-712 type hash for the Safe transaction type.
-     * @dev Precomputed value of: `keccak256("SafeTx(address to,uint256 value,bytes data,uint8 operation,uint256 safeTxGas,uint256 baseGas,uint256 gasPrice,address gasToken,address refundReceiver,uint256 nonce)")`.
+     * @dev The precomputed EIP-712 type hash for the Safe transaction type.
+     *      Precomputed value of: `keccak256("SafeTx(address to,uint256 value,bytes data,uint8 operation,uint256 safeTxGas,uint256 baseGas,uint256 gasPrice,address gasToken,address refundReceiver,uint256 nonce)")`.
      */
     bytes32 private constant SAFE_TX_TYPEHASH = 0xbb8310d486368db6bd6f849402fdd73ad53d316b5a4b2644ad6efe0f941286d8;
 
@@ -73,8 +73,8 @@ contract Safe is
     uint256 public override nonce;
 
     /**
-     * @notice Deprecated precomputed domain separator.
-     * @dev This deprecated storage variable is no longer in use but remains declared for storage layout compatibility across Safe versions.
+     * @dev Deprecated precomputed domain separator.
+     *      It is no longer in use but remains declared for storage layout compatibility across Safe versions.
      */
     bytes32 private _deprecatedDomainSeparator;
 

contracts/accessors/SimulateTxAccessor.sol

@@ -10,7 +10,7 @@ import {Executor, Enum} from "../base/Executor.sol";
  */
 contract SimulateTxAccessor is Executor {
     /**
-     * @notice The address of the {SimulateTxAccessor} contract.
+     * @dev The address of the {SimulateTxAccessor} contract.
      */
     address private immutable ACCESSOR_SINGLETON;
 

contracts/base/FallbackManager.sol

@@ -10,8 +10,8 @@ import {IFallbackManager} from "../interfaces/IFallbackManager.sol";
  */
 abstract contract FallbackManager is SelfAuthorized, IFallbackManager {
     /**
-     * @notice The storage slot used for storing the currently configured fallback handler address.
-     * @dev Precomputed value of: `keccak256("fallback_manager.handler.address")`.
+     * @dev The storage slot used for storing the currently configured fallback handler address.
+     *      Precomputed value of: `keccak256("fallback_manager.handler.address")`.
      */
     bytes32 internal constant FALLBACK_HANDLER_STORAGE_SLOT = 0x6c9a6c4a39284e37ed1cf53d337577d14212a4870fb976a4366c693b939918d5;
 

contracts/base/GuardManager.sol

@@ -70,8 +70,8 @@ abstract contract BaseTransactionGuard is ITransactionGuard {
  */
 abstract contract GuardManager is SelfAuthorized, IGuardManager {
     /**
-     * @notice The storage slot used for storing the currently configured transaction guard.
-     * @dev Precomputed value of: `keccak256("guard_manager.guard.address")`.
+     * @dev The storage slot used for storing the currently configured transaction guard.
+     *      Precomputed value of: `keccak256("guard_manager.guard.address")`.
      */
     bytes32 internal constant GUARD_STORAGE_SLOT = 0x4a204f620c8c5ccdca3fd54d003badd85ba500436a431f0cbda4f558c93c34c8;
 

contracts/base/ModuleManager.sol

@@ -63,22 +63,22 @@ abstract contract BaseModuleGuard is IModuleGuard {
  */
 abstract contract ModuleManager is SelfAuthorized, Executor, IModuleManager {
     /**
-     * @notice The sentinel module value in the {modules} linked list.
-     * @dev `SENTINEL_MODULES` is used to traverse {modules}, so that:
+     * @dev The sentinel module value in the {modules} linked list.
+     *      `SENTINEL_MODULES` is used to traverse {modules}, such that:
      *      1. `modules[SENTINEL_MODULES]` contains the first module
      *      2. `modules[last_module]` points back to `SENTINEL_MODULES`
      */
     address internal constant SENTINEL_MODULES = address(0x1);
 
     /**
-     * @notice The storage slot used for storing the currently configured module guard.
-     * @dev Precomputed value of: `keccak256("module_manager.module_guard.address")`.
+     * @dev The storage slot used for storing the currently configured module guard.
+     *      Precomputed value of: `keccak256("module_manager.module_guard.address")`.
      */
     bytes32 internal constant MODULE_GUARD_STORAGE_SLOT = 0xb104e0b93118902c651344349b610029d694cfdec91c589c91ebafbcd0289947;
 
     /**
-     * @notice The linked list of modules.
-     * @dev This mapping defines a linked list where while allowing for `O(1)` inclusion checks.
+     * @dev The linked list of modules, where `modules[module]` points to the next in the list.
+     *      A mapping is used to allow for `O(1)` inclusion checks.
      */
     mapping(address => address) internal modules;
 

contracts/base/OwnerManager.sol

@@ -13,26 +13,26 @@ import {IOwnerManager} from "../interfaces/IOwnerManager.sol";
  */
 abstract contract OwnerManager is SelfAuthorized, IOwnerManager {
     /**
-     * @notice The sentinel owner value in the {owners} linked list.
-     * @dev `SENTINEL_OWNERS` is used to traverse {owners}, so that:
+     * @dev The sentinel owner value in the {owners} linked list.
+     *      `SENTINEL_OWNERS` is used to traverse {owners}, such that:
      *      1. `owners[SENTINEL_OWNERS]` contains the first owner
      *      2. `owners[last_owner]` points back to `SENTINEL_OWNERS`
      */
     address internal constant SENTINEL_OWNERS = address(0x1);
 
     /**
-     * @notice The linked list of owners.
-     * @dev This mapping defines a linked list where while allowing for `O(1)` inclusion checks.
+     * @dev The linked list of owners, where `owners[owner]` points to the next in the list.
+     *      A mapping is used to allow for `O(1)` inclusion checks.
      */
     mapping(address => address) internal owners;
 
     /**
-     * @notice The number of owners.
+     * @dev The number of owners.
      */
     uint256 internal ownerCount;
 
     /**
-     * @notice The threshold of owners required to sign a transaction.
+     * @dev The threshold of owners required to sign a transaction.
      */
     uint256 internal threshold;
 

contracts/interfaces/IModuleManager.sol

@@ -5,7 +5,7 @@ import {Enum} from "../libraries/Enum.sol";
 /**
  * @title IModuleManager - An interface of contract managing Safe modules
  * @notice Modules are extensions with unlimited access to a Safe that can be added to a Safe by its owners.
-           ⚠️ WARNING: Modules are a security risk since they can execute arbitrary transactions, 
+           ⚠️ WARNING: Modules are a security risk since they can execute arbitrary transactions,
            so only trusted and audited modules should be added to a Safe. A malicious module can
            completely takeover a Safe.
  * @author @safe-global/safe-protocol

contracts/interfaces/ISafe.sol

@@ -176,6 +176,7 @@ interface ISafe is IModuleManager, IGuardManager, IOwnerManager, IFallbackManage
      * @notice Returns a descriptive version of the Safe contract.
      * @return The version string.
      */
+    // solhint-disable-next-line func-name-mixedcase
     function VERSION() external view returns (string memory);
 
     /**