Consider replacing Solhint with Slippy

contracts/governance/Governor.sol

@@ -792,7 +792,7 @@ abstract contract Governor is Context, ERC165, EIP712, Nonces, IGovernor, IERC72
 
     /// @inheritdoc IERC6372
     // solhint-disable-next-line func-name-mixedcase
-    function CLOCK_MODE() public view virtual returns (string memory);
+    function CLOCK_MODE() public view virtual returns (string memory); // slippy-disable-line naming-convention
 
     /// @inheritdoc IGovernor
     function votingDelay() public view virtual returns (uint256);

contracts/governance/IGovernor.sol

@@ -199,7 +199,7 @@ interface IGovernor is IERC165, IERC6372 {
      * JavaScript class.
      */
     // solhint-disable-next-line func-name-mixedcase
-    function COUNTING_MODE() external view returns (string memory);
+    function COUNTING_MODE() external view returns (string memory); // slippy-disable-line naming-convention
 
     /**
      * @notice module:core

contracts/governance/extensions/GovernorCountingFractional.sol

@@ -56,6 +56,7 @@ abstract contract GovernorCountingFractional is Governor {
     /// @inheritdoc IGovernor
     // solhint-disable-next-line func-name-mixedcase
     function COUNTING_MODE() public pure virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "support=bravo,fractional&quorum=for,abstain&params=fractional";
     }
 

contracts/governance/extensions/GovernorCountingOverridable.sol

@@ -50,6 +50,7 @@ abstract contract GovernorCountingOverridable is GovernorVotes {
     /// @inheritdoc IGovernor
     // solhint-disable-next-line func-name-mixedcase
     function COUNTING_MODE() public pure virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "support=bravo,override&quorum=for,abstain&overridable=true";
     }
 

contracts/governance/extensions/GovernorCountingSimple.sol

@@ -30,6 +30,7 @@ abstract contract GovernorCountingSimple is Governor {
     /// @inheritdoc IGovernor
     // solhint-disable-next-line func-name-mixedcase
     function COUNTING_MODE() public pure virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "support=bravo&quorum=for,abstain";
     }
 

contracts/governance/extensions/GovernorVotes.sol

@@ -43,6 +43,7 @@ abstract contract GovernorVotes is Governor {
      */
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         try token().CLOCK_MODE() returns (string memory clockmode) {
             return clockmode;
         } catch {

contracts/governance/utils/Votes.sol

@@ -64,6 +64,7 @@ abstract contract Votes is Context, EIP712, Nonces, IERC5805 {
      */
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         // Check that the clock was not modified
         if (clock() != Time.blockNumber()) {
             revert ERC6372InconsistentClock();

contracts/interfaces/IERC6372.sol

@@ -13,5 +13,5 @@ interface IERC6372 {
      * @dev Description of the clock
      */
     // solhint-disable-next-line func-name-mixedcase
-    function CLOCK_MODE() external view returns (string memory);
+    function CLOCK_MODE() external view returns (string memory); // slippy-disable-line naming-convention
 }

contracts/mocks/MultipleInheritanceInitializableMocks.sol

@@ -27,11 +27,13 @@ contract SampleHuman is Initializable {
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleHuman_init() internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         __SampleHuman_init_unchained();
     }
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleHuman_init_unchained() internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         isHuman = true;
     }
 }
@@ -48,12 +50,14 @@ contract SampleMother is Initializable, SampleHuman {
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleMother_init(uint256 value) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         __SampleHuman_init();
         __SampleMother_init_unchained(value);
     }
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleMother_init_unchained(uint256 value) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         mother = value;
     }
 }
@@ -70,12 +74,14 @@ contract SampleGramps is Initializable, SampleHuman {
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleGramps_init(string memory value) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         __SampleHuman_init();
         __SampleGramps_init_unchained(value);
     }
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleGramps_init_unchained(string memory value) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         gramps = value;
     }
 }
@@ -92,12 +98,14 @@ contract SampleFather is Initializable, SampleGramps {
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleFather_init(string memory _gramps, uint256 _father) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         __SampleGramps_init(_gramps);
         __SampleFather_init_unchained(_father);
     }
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleFather_init_unchained(uint256 _father) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         father = _father;
     }
 }
@@ -114,6 +122,7 @@ contract SampleChild is Initializable, SampleMother, SampleFather {
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleChild_init(
+        // slippy-disable-previous-line naming-convention
         uint256 _mother,
         string memory _gramps,
         uint256 _father,
@@ -126,6 +135,7 @@ contract SampleChild is Initializable, SampleMother, SampleFather {
 
     // solhint-disable-next-line func-name-mixedcase
     function __SampleChild_init_unchained(uint256 _child) internal onlyInitializing {
+        // slippy-disable-previous-line naming-convention
         child = _child;
     }
 }

contracts/mocks/Stateless.sol

@@ -2,6 +2,7 @@
 
 pragma solidity ^0.8.26;
 
+// slippy-disable no-unused-vars
 // We keep these imports and a dummy contract just to we can run the test suite after transpilation.
 
 import {Accumulators} from "../utils/structs/Accumulators.sol";

contracts/mocks/VotesExtendedMock.sol

@@ -37,6 +37,7 @@ abstract contract VotesExtendedTimestampMock is VotesExtendedMock {
 
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "mode=timestamp";
     }
 }

contracts/mocks/VotesMock.sol

@@ -37,6 +37,7 @@ abstract contract VotesTimestampMock is VotesMock {
 
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "mode=timestamp";
     }
 }

contracts/mocks/account/modules/ERC7579Mock.sol

@@ -77,7 +77,7 @@ abstract contract ERC7579FallbackHandlerMock is ERC7579ModuleMock(MODULE_TYPE_FA
         emit ERC7579FallbackHandlerMockCalled(_msgAccount(), _msgSender(), msg.value, _msgData());
     }
 
-    function callView() public view returns (address, address) {
+    function callView() public view returns (address account, address sender) {
         return (_msgAccount(), _msgSender());
     }
 

contracts/mocks/docs/governance/MyTokenTimestampBased.sol

@@ -17,6 +17,7 @@ contract MyTokenTimestampBased is ERC20, ERC20Permit, ERC20Votes {
 
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public pure override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "mode=timestamp";
     }
 

contracts/mocks/token/ERC20VotesAdditionalCheckpointsMock.sol

@@ -26,6 +26,7 @@ abstract contract ERC20VotesExtendedTimestampMock is ERC20VotesExtendedMock {
 
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "mode=timestamp";
     }
 }

contracts/mocks/token/ERC20VotesTimestampMock.sol

@@ -13,6 +13,7 @@ abstract contract ERC20VotesTimestampMock is ERC20Votes {
 
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "mode=timestamp";
     }
 }
@@ -24,6 +25,7 @@ abstract contract ERC721VotesTimestampMock is ERC721Votes {
 
     // solhint-disable-next-line func-name-mixedcase
     function CLOCK_MODE() public view virtual override returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return "mode=timestamp";
     }
 }

contracts/proxy/utils/UUPSUpgradeable.sol

@@ -20,7 +20,7 @@ import {ERC1967Utils} from "../ERC1967/ERC1967Utils.sol";
  */
 abstract contract UUPSUpgradeable is IERC1822Proxiable {
     /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
-    address private immutable __self = address(this);
+    address private immutable __self = address(this); // slippy-disable-line naming-convention
 
     /**
      * @dev The version of the upgrade interface of the contract. If this getter is missing, both `upgradeTo(address)`

contracts/token/ERC20/extensions/ERC20Permit.sol

@@ -72,6 +72,7 @@ abstract contract ERC20Permit is ERC20, IERC20Permit, EIP712, Nonces {
     /// @inheritdoc IERC20Permit
     // solhint-disable-next-line func-name-mixedcase
     function DOMAIN_SEPARATOR() external view returns (bytes32) {
+        // slippy-disable-previous-line naming-convention
         return _domainSeparatorV4();
     }
 }

contracts/token/ERC20/extensions/IERC20Permit.sol

@@ -86,5 +86,5 @@ interface IERC20Permit {
      * @dev Returns the domain separator used in the encoding of the signature for {permit}, as defined by {EIP712}.
      */
     // solhint-disable-next-line func-name-mixedcase
-    function DOMAIN_SEPARATOR() external view returns (bytes32);
+    function DOMAIN_SEPARATOR() external view returns (bytes32); // slippy-disable-line naming-convention
 }

contracts/utils/Packing.sol

@@ -33,6 +33,7 @@ pragma solidity ^0.8.20;
  * _Available since v5.1._
  */
 // solhint-disable func-name-mixedcase
+// slippy-disable naming-convention
 library Packing {
     error OutOfRangeAccess();
 

contracts/utils/cryptography/EIP712.sol

@@ -144,6 +144,7 @@ abstract contract EIP712 is IERC5267 {
      */
     // solhint-disable-next-line func-name-mixedcase
     function _EIP712Name() internal view returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return _name.toStringWithFallback(_nameFallback);
     }
 
@@ -155,6 +156,7 @@ abstract contract EIP712 is IERC5267 {
      */
     // solhint-disable-next-line func-name-mixedcase
     function _EIP712Version() internal view returns (string memory) {
+        // slippy-disable-previous-line naming-convention
         return _version.toStringWithFallback(_versionFallback);
     }
 }

contracts/vendor/compound/ICompoundTimelock.sol

@@ -38,13 +38,13 @@ interface ICompoundTimelock {
     receive() external payable;
 
     // solhint-disable-next-line func-name-mixedcase
-    function GRACE_PERIOD() external view returns (uint256);
+    function GRACE_PERIOD() external view returns (uint256); // slippy-disable-line naming-convention
 
     // solhint-disable-next-line func-name-mixedcase
-    function MINIMUM_DELAY() external view returns (uint256);
+    function MINIMUM_DELAY() external view returns (uint256); // slippy-disable-line naming-convention
 
     // solhint-disable-next-line func-name-mixedcase
-    function MAXIMUM_DELAY() external view returns (uint256);
+    function MAXIMUM_DELAY() external view returns (uint256); // slippy-disable-line naming-convention
 
     function admin() external view returns (address);
 

package.json

@@ -64,6 +64,7 @@
     "@openzeppelin/merkle-tree": "^1.0.7",
     "@openzeppelin/upgrade-safe-transpiler": "^0.4.1",
     "@openzeppelin/upgrades-core": "^1.20.6",
+    "@slippy-lint/slippy": "^0.1.2",
     "chai": "^4.2.0",
     "eslint": "^9.0.0",
     "eslint-config-prettier": "^10.0.0",

scripts/generate/templates/Packing.js

@@ -36,6 +36,7 @@ pragma solidity ^0.8.20;
  * _Available since v5.1._
  */
 // solhint-disable func-name-mixedcase
+// slippy-disable naming-convention
 `;
 
 const errors = `\