Merge pull request 'clan: Fix treefmt not applying mypy lints on clan_cli and clan_app' (#1777) from Qubasa/clan-core:Qubasa-main into main
This commit is contained in:
@@ -13,10 +13,16 @@
|
||||
treefmt.programs.deadnix.enable = true;
|
||||
|
||||
treefmt.programs.mypy.directories = {
|
||||
"pkgs/clan-cli".extraPythonPackages = self'.packages.clan-cli.testDependencies;
|
||||
"pkgs/clan-app".extraPythonPackages =
|
||||
# clan-app currently only exists on linux
|
||||
(self'.packages.clan-app.externalTestDeps or [ ]) ++ self'.packages.clan-cli.testDependencies;
|
||||
"pkgs/clan-cli" = {
|
||||
extraPythonPackages = self'.packages.clan-cli.testDependencies;
|
||||
modules = [ "clan_cli" ];
|
||||
};
|
||||
"pkgs/clan-app" = {
|
||||
extraPythonPackages =
|
||||
# clan-app currently only exists on linux
|
||||
(self'.packages.clan-app.externalTestDeps or [ ]) ++ self'.packages.clan-cli.testDependencies;
|
||||
modules = [ "clan_app" ];
|
||||
};
|
||||
};
|
||||
treefmt.programs.ruff.check = true;
|
||||
treefmt.programs.ruff.format = true;
|
||||
|
||||
@@ -40,6 +40,9 @@ no_implicit_optional = true
|
||||
module = "argcomplete.*"
|
||||
ignore_missing_imports = true
|
||||
|
||||
[[tool.mypy.overrides]]
|
||||
module = "clan_cli.*"
|
||||
ignore_missing_imports = true
|
||||
|
||||
[tool.ruff]
|
||||
target-version = "py312"
|
||||
|
||||
@@ -8,6 +8,7 @@ from collections.abc import Callable
|
||||
from graphlib import TopologicalSorter
|
||||
from pathlib import Path
|
||||
from tempfile import TemporaryDirectory
|
||||
from typing import Any
|
||||
|
||||
from clan_cli.cmd import run
|
||||
|
||||
@@ -71,7 +72,7 @@ def decrypt_dependencies(
|
||||
) -> dict[str, dict[str, bytes]]:
|
||||
generator = machine.vars_generators[generator_name]
|
||||
dependencies = set(generator["dependencies"])
|
||||
decrypted_dependencies = {}
|
||||
decrypted_dependencies: dict[str, Any] = {}
|
||||
for dep_generator in dependencies:
|
||||
decrypted_dependencies[dep_generator] = {}
|
||||
dep_files = machine.vars_generators[dep_generator]["files"]
|
||||
@@ -242,6 +243,7 @@ def _generate_vars_for_machine(
|
||||
# process generators in topological order
|
||||
sorter = TopologicalSorter(graph)
|
||||
for generator_name in sorter.static_order():
|
||||
assert generator_name is not None
|
||||
machine_updated |= execute_generator(
|
||||
machine=machine,
|
||||
generator_name=generator_name,
|
||||
|
||||
Reference in New Issue
Block a user