From e432351fd29c901b0e7035d5dc246e162e5a0af0 Mon Sep 17 00:00:00 2001 From: Leandro Lucarella Date: Tue, 2 Jan 2024 15:09:42 +0100 Subject: [PATCH 1/2] mypy: Check also the `frequenz.client.weather` package Signed-off-by: Leandro Lucarella --- pyproject.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index 3a41c3d..8241131 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -148,7 +148,7 @@ namespace_packages = true # used but getting the original ignored error when removing the type: ignore. # See for example: https://github.com/python/mypy/issues/2960 #no_incremental = true -packages = ["frequenz.api.weather"] +packages = ["frequenz.api.weather", "frequenz.client.weather"] strict = true [[tool.mypy.overrides]] From e1136510b28a432a006771f35b9dc1034d996cea Mon Sep 17 00:00:00 2001 From: Leandro Lucarella Date: Tue, 2 Jan 2024 15:16:23 +0100 Subject: [PATCH 2/2] Run other linters to the client code We do so by customizing the noxfile repo config to include the `py/frequenz/client` directory as a source path. Signed-off-by: Leandro Lucarella --- noxfile.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/noxfile.py b/noxfile.py index 4690702..a790e79 100644 --- a/noxfile.py +++ b/noxfile.py @@ -3,6 +3,9 @@ """Configuration file for nox.""" -from frequenz.repo.config import RepositoryType, nox +from frequenz.repo.config import nox +from frequenz.repo.config.nox import default -nox.configure(RepositoryType.API) +config = default.api_config.copy() +config.source_paths = ["py/frequenz/client"] +nox.configure(config)