ruff improvements

Author: sanderegg

.ruff.toml

@@ -45,4 +45,27 @@ ignore = [
     "S101", # use of `assert` detected hanbled by pylance, does not support noseq
     "TID252", # [*] Relative imports from parent modules are banned
 ]
-extend-exclude = ["**/tests", "**/generated_models", "**/director/"]
+
+target-version = "py310"
+
+
+[per-file-ignores]
+"**/tests/**" = [
+  "T201", # print found
+  "ARG001", # unused function argument
+  "PT019", # user pytest.mark.usefixture
+  "PLR2004", # use of magic values
+  "PLR0913", # too many arguments
+  "N806", # Uppercase variables in functions
+  "PT001", # use pytest.fixture over pytest.fixture() whatsoever
+  "ERA001", # found commented out code
+]
+
+[flake8-pytest-style]
+fixture-parentheses = false
+parametrize-names-type = "csv"
+
+
+
+[pylint]
+max-args = 10

Makefile

@@ -514,6 +514,11 @@ ruff: ## python ruffing
 	@ruff --version
 	# Running ruff in packages and services (except director)
 	@ruff check \
+		--config=$(CURDIR)/.ruff.toml \
+		--respect-gitignore \
+		--extend-exclude="**/tests" \
+		--extend-exclude="**/generated_models" \
+		--extend-exclude="**/director/" \
 		$(CURDIR)/services \
 		$(CURDIR)/packages
 

scripts/common.Makefile

@@ -129,6 +129,13 @@ pyupgrade: ## upgrades python syntax for newer versions of the language (SEE htt
 pylint: $(REPO_BASE_DIR)/.pylintrc ## runs pylint (python linter) on src and tests folders
 	@pylint --rcfile="$(REPO_BASE_DIR)/.pylintrc" -v $(CURDIR)/src $(CURDIR)/tests
 
+.PHONY: ruff
+ruff: $(REPO_BASE_DIR)/.ruff.toml ## runs ruff (python fast linter) on src and tests folders
+	@ruff check \
+		--config=$(REPO_BASE_DIR)/.ruff.toml \
+		--respect-gitignore \
+		$(CURDIR)/src \
+		$(CURDIR)/tests
 
 .PHONY: mypy
 mypy: $(REPO_BASE_DIR)/scripts/mypy.bash $(REPO_BASE_DIR)/mypy.ini ## runs mypy python static type-checker on this services's code. Use AFTER make install-*