Added separate pylint run

Test: Added a separate pylint run (in addition to the one in the Ansible sanity test), because it can be run on the test sources as well, and because the santy test pylint has important checks disabled. (issue #1007) Details: * Added a new make target 'pylint'. * Added 'make pylnt' to the test workflow. * Fixed issues reported by pylint (many). Signed-off-by: Andreas Maier <[email protected]>
zhmcclient · Jul 3, 2024 · df3c585 · df3c585
1 parent 9870a29
commit df3c585
Show file tree

Hide file tree

Showing 62 changed files with 1,996 additions and 1,217 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -195,6 +195,12 @@ jobs:
         RUN_TYPE: ${{ steps.set-run-type.outputs.result }}
       run: |
         make check
+    - name: Run pylint
+      env:
+        PACKAGE_LEVEL: ${{ matrix.package_level }}
+        RUN_TYPE: ${{ steps.set-run-type.outputs.result }}
+      run: |
+        make pylint
     - name: Run sanity
       env:
         PACKAGE_LEVEL: ${{ matrix.package_level }}