Merge branch 'gh-actions'

ranger · Aug 18, 2023 · 0e9ca52 · 0e9ca52
2 parents 69a3d21 + f3e93c7
commit 0e9ca52
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 44 deletions.
diff --git a/.github/workflows/other.yml b/.github/workflows/other.yml
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -3,7 +3,9 @@ name: PR Pylint, PEP8 and Pytest
 on: pull_request
 
 jobs:
-  test_pylint:
+  test_PR:
+    env:
+      TERM: xterm
     runs-on: ubuntu-latest
     strategy:
       max-parallel: 4
@@ -25,22 +27,47 @@ jobs:
       with:
         list-files: 'shell'
         filters: |
+          this-workflow:
+            - modified: '.github/workflows/pr.yml'
           code:
             - added|modified: '!(ranger/config)**.py'
           config:
             - added|modified: 'ranger/config/**.py'
-    - name: Lint code with Flake8 and Pylint
-      if: ${{ steps.path-filter.outputs.code == 'true' }}
+    - name: Lint changed code with Flake8 and Pylint
+      if: steps.paths-filter.outputs.code == 'true'
+       && steps.paths-filter.outputs.this-workflow == 'false'
       run: |
         flake8 ${{ steps.paths-filter.outputs.code_files }}
         pylint ${{ steps.paths-filter.outputs.code_files }}
-    - name: Lint config with Flake8 and Pylint
-      if: ${{ steps.path-filter.outputs.config == 'true' }}
+    - name: Lint all code with Flake8 and Pylint
+      if: steps.paths-filter.outputs.this-workflow == 'true'
+      run: |
+        make TEST_PATHS_CONFIG= test_flake8
+        make TEST_PATHS_CONFIG=--version test_pylint
+    - name: Lint changed config with Flake8 and Pylint
+      if: steps.paths-filter.outputs.config == 'true'
+       && steps.paths-filter.outputs.this-workflow == 'false'
       run: |
         flake8 ${{ steps.paths-filter.outputs.config }}
         pylint --rcfile=ranger/config/.pylintrc \
           ${{ steps.paths-filter.outputs.config_files }}
+    - name: Lint all config with Flake8 and Pylint
+      if: steps.paths-filter.outputs.this-workflow == 'true'
+      run: |
+        make TEST_PATHS_MAIN= test_flake8
+        make TEST_PATHS_MAIN=--version test_pylint
+    - name: Run doctest
+      if: steps.paths-filter.outputs.code == 'true'
+       || steps.paths-filter.outputs.this-workflow == 'true'
+      run: |
+        make test_doctest
     - name: Test code with Pytest
-      if: ${{ steps.path-filter.outputs.code == 'true' }}
+      if: steps.paths-filter.outputs.code == 'true'
+       || steps.paths-filter.outputs.this-workflow == 'true'
       run: |
         make test_pytest
+    - name: Check man page completeness
+      if: steps.paths-filter.outputs.code == 'true'
+       || steps.paths-filter.outputs.this-workflow == 'true'
+      run: |
+        make test_other
diff --git a/.github/workflows/pylint.yml → .github/workflows/python.yml b/.github/workflows/pylint.yml → .github/workflows/python.yml
@@ -1,4 +1,4 @@
-name: Pylint, PEP8 and Pytest
+name: Python linting and tests
 
 on:
   push:
@@ -26,4 +26,4 @@ jobs:
         pip install -r requirements.txt
     - name: Lint with Flake8 and Pylint, test with Pytest
       run: |
-        make test_flake8 test_pylint test_pytest
+        make test_py