Add some more mypy checks

Author: sbrunner

.prospector.yml

@@ -14,8 +14,15 @@ mypy:
   options:
     ignore-missing-imports: true
     follow-imports: skip
+    check-untyped-defs: true
+    disallow-any-generics: true
     disallow-untyped-defs: true
-    #strict: true
+    disallow-incomplete-defs: true
+    disallow-untyped-decorators: true
+    warn-unused-configs: true
+    warn-unused-ignores: true
+    warn-redundant-casts: true
+    extra-checks: true
 
 pylint:
   options:

prospector/config/__init__.py

@@ -98,10 +98,11 @@ def replace_deprecated_tool_names(self) -> list[str]:
 
     def get_output_report(self) -> list[tuple[str, list[str]]]:
         # Get the output formatter
+        output_report: list[tuple[str, list[str]]]
         if self.config.output_format is not None:
             output_report = self.config.output_format
         else:
-            output_report = [(self.profile.output_format, self.profile.output_target)]
+            output_report = [(self.profile.output_format, self.profile.output_target)]  # type: ignore[list-item]
 
         for index, report in enumerate(output_report):
             if not all(report):
@@ -262,7 +263,7 @@ def _determine_tool_runners(self, config: setoptconf.config.Configuration, profi
 
     def _determine_ignores(
         self, config: setoptconf.config.Configuration, profile: ProspectorProfile, libraries: list[str]
-    ) -> list[re.Pattern]:
+    ) -> list[re.Pattern[str]]:
         # Grab ignore patterns from the options
         ignores = []
         for pattern in config.ignore_patterns + profile.ignore_patterns:

prospector/finder.py

@@ -18,7 +18,7 @@ class FileFinder:
     is basically to know which files to pass to which tools to be inspected.
     """
 
-    def __init__(self, *provided_paths: Path, exclusion_filters: Optional[Iterable[Callable]] = None):
+    def __init__(self, *provided_paths: Path, exclusion_filters: Optional[Iterable[Callable[[Path], bool]]] = None):
         """
         :param provided_paths:
             A list of Path objects to search for files and modules - can be either directories or files

prospector/profiles/profile.py

@@ -58,7 +58,7 @@ def get_disabled_messages(self, tool_name: str) -> list[str]:
         return list(set(disable) - set(enable))
 
     def is_tool_enabled(self, name: str) -> bool:
-        enabled = getattr(self, name).get("run")
+        enabled: Optional[bool] = getattr(self, name).get("run")
         if enabled is not None:
             return enabled
         # this is not explicitly enabled or disabled, so use the default

prospector/tools/mypy/__init__.py

@@ -45,7 +45,9 @@ def format_message(message: str) -> Message:
     )
 
 
-def _run_in_subprocess(q: Queue, cmd: Callable[[list[str]], tuple[str, str]], paths: list[str]) -> None:
+def _run_in_subprocess(
+    q: "Queue[tuple[str, str]]", cmd: Callable[[list[str]], tuple[str, str]], paths: list[str]
+) -> None:
     """
     This function exists only to be called by multiprocessing.Process as using
     lambda is forbidden

prospector/tools/profile_validator/__init__.py

@@ -62,7 +62,7 @@ def __init__(self) -> None:
         self.ignore_codes: list[str] = []
 
     def configure(self, prospector_config: "ProspectorConfig", found_files: FileFinder) -> None:
-        for profile in prospector_config.config.profiles:  # type: ignore[attr-defined]
+        for profile in prospector_config.config.profiles:
             self.to_check.add(profile)
 
         self.ignore_codes = prospector_config.get_disabled_messages("profile-validator")