style: simplify static analysis of python files

Author: vil02

.github/workflows/python_test.yml

@@ -36,13 +36,9 @@ jobs:
         run: |
           poetry install --with dev
 
-      - name: Lint with ruff
-        run: |
-          poetry run ruff check
-
       - name: Run liter
         run: |
-          ./check_all_python_scripts.sh
+          ./check_all_python_files.sh
 
       - name: Install language dependencies
         working-directory: ${{github.workspace}}/system_setup_scripts

check_all_python_scripts.sh check_all_python_files.shcheck_all_python_scripts.sh renamed to check_all_python_files.sh

@@ -2,6 +2,8 @@
 
 set -euo pipefail
 
+poetry run ruff check .
+poetry run mypy .
 poetry run bandit -c bandit.yml -r .
 
 find . -name "*.py" -not -path "./tests/example_data/python3/*" -exec ./check_python_file.sh {} +

check_python_file.sh

@@ -7,20 +7,20 @@ declare -i exit_code=0
 for cur_file in "$@"
 do
     printf "Checking \"%s\"\n" "${cur_file}"
-    printf "Checking with pylint:\n"
-    if ! poetry run pylint "${cur_file}" ; then
+    pylint_output=$(poetry run pylint "${cur_file}" 2>&1) || {
+        printf "Checking with pylint:\n%s\n" "${pylint_output}"
         exit_code=1
-    fi
+    }
 
-    printf "Checking with flake8:\n"
-    if ! poetry run flake8 "${cur_file}" --count --max-line-length=88 --show-source --ignore=E203; then
+    flake8_output=$(poetry run flake8 "${cur_file}" --count --max-line-length=88 --show-source --ignore=E203,W503 2>&1) || {
+        printf "Checking with flake8:\n%s\n" "${flake8_output}"
         exit_code=1
-    fi
+    }
 
-    printf "Checking with mypy:\n"
-    if ! poetry run mypy "${cur_file}"; then
+    mypy_output=$(poetry run mypy "${cur_file}" 2>&1) || {
+        printf "Checking with mypy:\n%s\n" "${mypy_output}"
         exit_code=1
-    fi
+    }
 done
 
 if [[ ${exit_code} -eq 0 ]] ; then