Sbachmei/mic 5399/mypy errors logging manager (#531)

CHANGELOG.rst

@@ -1,3 +1,7 @@
+**3.0.18 - TBD/TBD/TBD**
+
+  - Fix mypy errors in vivarium/framework/logging/manager.py
+
 **3.0.17 - 11/04/24**
 
   - Fix mypy errors in vivarium/framework/configuration.py

pyproject.toml

@@ -48,7 +48,6 @@ exclude = [
     'src/vivarium/framework/components/manager.py',
     'src/vivarium/framework/components/parser.py',
     'src/vivarium/framework/engine.py',
-    'src/vivarium/framework/logging/manager.py',
     'src/vivarium/framework/lookup/manager.py',
     'src/vivarium/framework/population/manager.py',
     'src/vivarium/framework/population/population_view.py',

src/vivarium/framework/logging/manager.py

@@ -1,10 +1,10 @@
-# mypy: ignore-errors
 """
 =====================
 The Logging Subsystem
 =====================
 
 """
+
 from __future__ import annotations
 
 import loguru
@@ -15,8 +15,8 @@
 
 
 class LoggingManager(Manager):
-    def __init__(self):
-        self._simulation_name = None
+    def __init__(self) -> None:
+        self._simulation_name: str | None = None
 
     def configure_logging(
         self,
@@ -39,22 +39,22 @@ def _terminal_logging_not_configured() -> bool:
         # fragile since it depends on a loguru's internals as well as the stability of code
         # paths in vivarium, but both are quite stable at this point, so I think it's pretty,
         # low risk.
-        return 1 not in logger._core.handlers
+        return 1 not in logger._core.handlers  # type: ignore[attr-defined]
 
     @property
-    def name(self):
+    def name(self) -> str:
         return "logging_manager"
 
-    def get_logger(self, component_name: str = None) -> loguru.Logger:
+    def get_logger(self, component_name: str | None = None) -> loguru.Logger:
         bind_args = {"simulation": self._simulation_name}
         if component_name:
             bind_args["component"] = component_name
         return logger.bind(**bind_args)
 
 
 class LoggingInterface(Interface):
-    def __init__(self, manager: LoggingManager):
+    def __init__(self, manager: LoggingManager) -> None:
         self._manager = manager
 
-    def get_logger(self, component_name: str = None) -> loguru.Logger:
+    def get_logger(self, component_name: str | None = None) -> loguru.Logger:
         return self._manager.get_logger(component_name)