fix some warnings in automation contracts (#11867)

smartcontractkit · Jan 24, 2024 · cb785bb · cb785bb
1 parent d627fe9
commit cb785bb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/contracts/src/v0.8/automation/dev/MercuryRegistry.sol b/contracts/src/v0.8/automation/dev/MercuryRegistry.sol
@@ -1,9 +1,9 @@
 pragma solidity 0.8.6;
 
-import "../../shared/access/ConfirmedOwner.sol";
-import "../interfaces/AutomationCompatibleInterface.sol";
-import "../interfaces/StreamsLookupCompatibleInterface.sol";
-import "../../ChainSpecificUtil.sol";
+import {ConfirmedOwner} from "../../shared/access/ConfirmedOwner.sol";
+import {AutomationCompatibleInterface} from "../interfaces/AutomationCompatibleInterface.sol";
+import {StreamsLookupCompatibleInterface} from "../interfaces/StreamsLookupCompatibleInterface.sol";
+import {ChainSpecificUtil} from "../../ChainSpecificUtil.sol";
 
 /*--------------------------------------------------------------------------------------------------------------------+
 | Mercury + Automation                                                                                                |

diff --git a/contracts/src/v0.8/automation/dev/MercuryRegistryBatchUpkeep.sol b/contracts/src/v0.8/automation/dev/MercuryRegistryBatchUpkeep.sol
@@ -1,20 +1,20 @@
 pragma solidity 0.8.6;
 
-import "../../shared/access/ConfirmedOwner.sol";
-import "../interfaces/AutomationCompatibleInterface.sol";
-import "../interfaces/StreamsLookupCompatibleInterface.sol";
-import "./MercuryRegistry.sol";
+import {ConfirmedOwner} from "../../shared/access/ConfirmedOwner.sol";
+import {AutomationCompatibleInterface} from "../interfaces/AutomationCompatibleInterface.sol";
+import {StreamsLookupCompatibleInterface} from "../interfaces/StreamsLookupCompatibleInterface.sol";
+import {MercuryRegistry} from "./MercuryRegistry.sol";
 
 contract MercuryRegistryBatchUpkeep is ConfirmedOwner, AutomationCompatibleInterface, StreamsLookupCompatibleInterface {
   error BatchSizeTooLarge(uint256 batchsize, uint256 maxBatchSize);
   // Use a reasonable maximum batch size. Every Mercury report is ~750 bytes, too many reports
   // passed into a single batch could exceed the calldata or transaction size limit for some blockchains.
-  uint256 constant MAX_BATCH_SIZE = 50;
+  uint256 public constant MAX_BATCH_SIZE = 50;
 
-  MercuryRegistry immutable i_registry; // master registry, where feed data is stored
+  MercuryRegistry public immutable i_registry; // master registry, where feed data is stored
 
-  uint256 s_batchStart; // starting index of upkeep batch on the MercuryRegistry's s_feeds array, inclusive
-  uint256 s_batchEnd; // ending index of upkeep batch on the MercuryRegistry's s_feeds array, exclusive
+  uint256 public s_batchStart; // starting index of upkeep batch on the MercuryRegistry's s_feeds array, inclusive
+  uint256 public s_batchEnd; // ending index of upkeep batch on the MercuryRegistry's s_feeds array, exclusive
 
   constructor(address mercuryRegistry, uint256 batchStart, uint256 batchEnd) ConfirmedOwner(msg.sender) {
     i_registry = MercuryRegistry(mercuryRegistry);