Merge pull request #1039 from graphprotocol/fix_oz_n-09

fix: add override keyword in functions (OZ N-09)
graphprotocol · Sep 20, 2024 · f2a5498 · f2a5498
2 parents 95b3f83 + 516a4e3
commit f2a5498
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/horizon/contracts/data-service/extensions/DataServicePausable.sol b/packages/horizon/contracts/data-service/extensions/DataServicePausable.sol
@@ -30,14 +30,14 @@ abstract contract DataServicePausable is Pausable, DataService, IDataServicePaus
     /**
      * @notice See {IDataServicePausable-pause}
      */
-    function pause() external onlyPauseGuardian whenNotPaused {
+    function pause() external override onlyPauseGuardian whenNotPaused {
         _pause();
     }
 
     /**
      * @notice See {IDataServicePausable-pause}
      */
-    function unpause() external onlyPauseGuardian whenPaused {
+    function unpause() external override onlyPauseGuardian whenPaused {
         _unpause();
     }
 

diff --git a/packages/horizon/contracts/data-service/extensions/DataServicePausableUpgradeable.sol b/packages/horizon/contracts/data-service/extensions/DataServicePausableUpgradeable.sol
@@ -26,14 +26,14 @@ abstract contract DataServicePausableUpgradeable is PausableUpgradeable, DataSer
     /**
      * @notice See {IDataServicePausable-pause}
      */
-    function pause() external onlyPauseGuardian whenNotPaused {
+    function pause() external override onlyPauseGuardian whenNotPaused {
         _pause();
     }
 
     /**
      * @notice See {IDataServicePausable-pause}
      */
-    function unpause() external onlyPauseGuardian whenPaused {
+    function unpause() external override onlyPauseGuardian whenPaused {
         _unpause();
     }
 

diff --git a/packages/subgraph-service/contracts/SubgraphService.sol b/packages/subgraph-service/contracts/SubgraphService.sol
@@ -351,7 +351,7 @@ contract SubgraphService is
     /**
      * @notice See {ISubgraphService.setRewardsDestination}
      */
-    function setRewardsDestination(address rewardsDestination) external {
+    function setRewardsDestination(address rewardsDestination) external override {
         _setRewardsDestination(msg.sender, rewardsDestination);
     }