Ignoring code coverage on type checking (#1872)

armi/plugins.py

@@ -125,10 +125,7 @@
 from armi import pluginManager
 from armi.utils import flags
 
-# Not used during runtime so we could have a coverage drop here. Add the
-# pragma line to tell coverage.py to skip this
-# https://coverage.readthedocs.io/en/stable/excluding.html
-if TYPE_CHECKING:  # pragma: no cover
+if TYPE_CHECKING:
     from armi.reactor.composites import Composite
 
 

armi/reactor/__init__.py

@@ -55,11 +55,7 @@
 
 from armi import plugins
 
-# Provide type checking but avoid circular imports
-# Not used during runtime so we could have a coverage drop here. Add the
-# pragma line to tell coverage.py to skip this
-# https://coverage.readthedocs.io/en/stable/excluding.html
-if TYPE_CHECKING:  # pragma: no cover
+if TYPE_CHECKING:
     from armi.reactor.reactors import Core
     from armi.reactor.assemblyLists import SpentFuelPool
 

pyproject.toml

@@ -249,6 +249,9 @@ exclude_also = [
     "def __repr__",
     "if self\\.debug",
 
+    # Don't complain about missing type checking-only code:
+    "if TYPE_CHECKING",
+
     # Don't complain if tests don't hit defensive assertion code:
     "raise AssertionError",
     "raise KeyboardInterrupt",