f: lint warnings

Author: matiasedgeandnode

packages/horizon/contracts/data-service/utilities/ProvisionManager.sol

@@ -130,12 +130,6 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
         _requireValidProvision(serviceProvider);
     }
 
-    function _requireValidProvision(address serviceProvider) internal view {
-        IHorizonStaking.Provision memory provision = _getProvision(serviceProvider);
-        _checkProvisionTokens(provision);
-        _checkProvisionParameters(provision, false);
-    }
-
     /**
      * @notice Initializes the contract and any parent contracts.
      */
@@ -213,6 +207,12 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
         emit ThawingPeriodRangeSet(_min, _max);
     }
 
+    function _requireValidProvision(address _serviceProvider) internal view {
+        IHorizonStaking.Provision memory provision = _getProvision(_serviceProvider);
+        _checkProvisionTokens(provision);
+        _checkProvisionParameters(provision, false);
+    }
+
     // -- checks --
 
     /**

packages/subgraph-service/contracts/SubgraphService.sol

@@ -515,10 +515,6 @@ contract SubgraphService is
         return address(_graphStaking());
     }
 
-    function _requireRegisteredIndexer(address indexer) internal view {
-        require(indexers[indexer].registeredAt != 0, SubgraphServiceIndexerNotRegistered(indexer));
-    }
-
     function _cancelAllocationIndexingAgreement(address _allocationId) internal {
         IndexingAgreement._getManager().cancelForAllocation(_allocationId);
     }
@@ -534,6 +530,10 @@ contract SubgraphService is
         emit PaymentsDestinationSet(_indexer, _paymentsDestination);
     }
 
+    function _requireRegisteredIndexer(address _indexer) internal view {
+        require(indexers[_indexer].registeredAt != 0, SubgraphServiceIndexerNotRegistered(_indexer));
+    }
+
     // -- Data service parameter getters --
     /**
      * @notice Getter for the accepted thawing period range for provisions