Add pylint and ruff linter in the make lint-python target and point make lint

to the renamed style-check-python and lint-python targets.

Author: jonasbardino

Makefile

@@ -52,14 +52,20 @@ ifneq ($(MIG_ENV),'local')
 	@echo "unavailable outside local development environment"
 	@exit 1
 endif
+	@make style-check-python
 	@make lint-python
 
 # NOTE: black and isort use pyproject.toml to temporarily exclude a few paths
-.PHONY: lint-python
-lint-python:
+.PHONY: style-check-python
+style-check-python:
 	@$(LOCAL_PYTHON_BIN) -m black $(LINT_ENFORCE_DIRS) --check
 	@$(LOCAL_PYTHON_BIN) -m isort $(LINT_ENFORCE_DIRS) --check-only
 
+.PHONY: lint-python
+lint-python:
+	@$(LOCAL_PYTHON_BIN) -m pylint $(LINT_ENFORCE_DIRS) --errors-only
+	@$(LOCAL_PYTHON_BIN) -m ruff check $(LINT_ENFORCE_DIRS)
+
 .PHONY: clean
 clean:
 	@rm -f ./envhelp/py3.imageid

local-requirements.txt

@@ -5,6 +5,8 @@
 autopep8;python_version >= "3"
 black
 isort
+pylint
+ruff
 # We need paramiko for the ssh unit tests
 # NOTE: paramiko-3.0.0 dropped python2 and python3.6 support
 paramiko;python_version >= "3.7"

pyproject.toml

@@ -23,6 +23,7 @@ extend-exclude = '''
 [tool.isort]
 profile = "black"
 line_length = 80
+skip_glob = [".git/*", "tests/data/*", "tests/fixture/*"]
 # NOTE: the following paths must be kept in sync with black extend-exclude
 skip = [
     "bin/checkconf.py",
@@ -33,4 +34,34 @@ skip = [
     "sbin/grid_sftp.py",
     "sbin/grid_webdavs.py"
 ]
-skip_glob = [".git/*", "tests/data/*", "tests/fixture/*"]
+
+[tool.pylint]
+max-line-length = 80
+ignore-pattern = '''
+      tests/data/
+    | tests/fixture/
+'''
+# NOTE: the following paths must be kept in sync with black extend-exclude
+ignore-paths = [
+    "bin/checkconf.py",
+    "bin/createresource.py",
+    "bin/notifypassword.py",
+    "sbin/grid_ftps.py",
+    "sbin/grid_openid.py",
+    "sbin/grid_sftp.py",
+    "sbin/grid_webdavs.py"
+]
+
+[tool.ruff]
+line-length = 80
+exclude = [".git/", "tests/data/", "tests/fixture/"]
+# NOTE: the following regex matches must be kept in sync with isort skip
+extend-exclude = [
+    "bin/checkconf.py",
+    "bin/createresource.py",
+    "bin/notifypassword.py",
+    "sbin/grid_ftps.py",
+    "sbin/grid_openid.py",
+    "sbin/grid_sftp.py",
+    "sbin/grid_webdavs.py"
+]