refactor: required confs

src/ape/api/providers.py

@@ -1535,9 +1535,9 @@ def poll_logs(
         required_confirmations: Optional[int] = None,
         new_block_timeout: Optional[int] = None,
     ) -> Iterator[ContractLog]:
-        required_confirmations = (
-            required_confirmations or self.provider.network.required_confirmations
-        )
+        if required_confirmations is None:
+            required_confirmations = self.network.required_confirmations
+
         for block in self.poll_blocks(stop_block, required_confirmations, new_block_timeout):
             if block.number is None:
                 raise ValueError("Block number cannot be None")

src/ape/contracts/base.py

@@ -688,9 +688,8 @@ def poll_logs(
             Iterator[:class:`~ape.types.ContractLog`]
         """
 
-        required_confirmations = (
-            required_confirmations or self.provider.network.required_confirmations
-        )
+        if required_confirmations is None:
+            required_confirmations = self.provider.network.required_confirmations
 
         # NOTE: We process historical blocks separately here to minimize rpc calls
         height = max(self.chain_manager.blocks.height - required_confirmations, 0)

src/ape/managers/chain.py

@@ -269,6 +269,7 @@ def poll_blocks(
         Returns:
             Iterator[:class:`~ape.api.providers.BlockAPI`]
         """
+
         if required_confirmations is None:
             required_confirmations = self.network_confirmations