resolve conversation

Author: Jannis-Mittenzwei

exasol/toolbox/nox/_lint.py

@@ -95,7 +95,7 @@ def find_illegal(part) -> list[str]:
                 if _source_filter(version)
             ]
 
-        illegal: dict[str, list[str]] = {}
+        illegal: Dict[str, List[str]] = {}
         toml = tomlkit.loads(pyproject_toml)
         poetry = toml.get("tool", {}).get("poetry", {})
 
@@ -115,11 +115,11 @@ def find_illegal(part) -> list[str]:
         return Dependencies(illegal)
 
     @property
-    def illegal(self) -> dict[str, list[str]]:
+    def illegal(self) -> Dict[str, List[str]]:
         return self._illegal
 
 
-def report_illegal(illegal: dict[str, list[str]], console: rich.console.Console):
+def report_illegal(illegal: Dict[str, List[str]], console: rich.console.Console):
     count = sum(len(deps) for deps in illegal.values())
     suffix = "y" if count == 1 else "ies"
     console.print(f"{count} illegal dependenc{suffix}\n", style="red")
@@ -148,7 +148,7 @@ def type_check(session: Session) -> None:
 def security_lint(session: Session) -> None:
     """Runs the security linter on the project"""
     py_files = [f"{file}" for file in python_files(PROJECT_CONFIG.root)]
-    _security_lint(session, py_files)
+    _security_lint(session, list(filter(lambda file: "test" not in file, py_files)))
 
 
 @nox.session(name="lint:dependencies", python=False)
@@ -159,35 +159,4 @@ def dependency_check(session: Session) -> None:
     console = rich.console.Console()
     if illegal := dependencies.illegal:
         report_illegal(illegal, console)
-        sys.exit(1)
-
-
-@nox.session(name="lint:import", python=False)
-def import_lint(session: Session) -> None:
-    """(experimental) Runs import linter on the project"""
-    parser = argparse.ArgumentParser(
-        usage="nox -s import-lint -- [options]",
-        description="Runs the import linter on the project",
-    )
-    parser.add_argument(
-        "-c",
-        "--config",
-        type=str,
-        help="path to the configuration file for the importlinter",
-        metavar="TEXT",
-    )
-
-    args: argparse.Namespace = parser.parse_args(args=session.posargs)
-    file: str = args.config
-    path: Path | None = None
-    if file is None:
-        path = getattr(
-            PROJECT_CONFIG, "import_linter_config", Path(".import_linter_config")
-        )
-    else:
-        path = Path(file)
-    if not path.exists():
-        session.error(
-            "Please make sure you have a configuration file for the importlinter"
-        )
-    _import_lint(session=session, path=path)
+        sys.exit(1)