Merge branch 'Inria-Empenn:main' into main

.github/workflows/code_quality.yml

@@ -8,9 +8,11 @@ on:
   push:
     paths:
       - '**.py'
+      - '.github/workflows/code_quality.yml'
   pull_request:
     paths:
       - '**.py'
+      - '.github/workflows/code_quality.yml'
 
 # Jobs that define the workflow
 jobs:
@@ -33,22 +35,23 @@ jobs:
     - uses: actions/cache@v3
       with:
         path: ~/.cache/pip
-        key: ${{ runner.os }}-pip-pylint
+        key: ${{ runner.os }}-pip-${{ hashFiles('setup.py') }}
         restore-keys: |
-          ${{ runner.os }}-pip-pylint
+          ${{ runner.os }}-pip-
 
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install pylint
+        pip install .[tests]
 
     - name: Analyse the code with pylint
       run: |
-        pylint --exit-zero narps_open > pylint_report_narps_open.txt
-        pylint --exit-zero tests > pylint_report_tests.txt
+        pylint --fail-under 8 --ignore-paths narps_open/pipelines/ narps_open > pylint_report_narps_open.txt
+        pylint --fail-under 8 tests > pylint_report_tests.txt
 
     - name: Archive pylint results
       uses: actions/upload-artifact@v3
+      if: failure() # Only if previous step failed
       with:
         name: pylint-reports-python
         path: |

.github/workflows/pipeline_tests.yml

@@ -47,7 +47,7 @@ jobs:
         echo "tests=$test_files" >> $GITHUB_OUTPUT
         echo "teams=$teams" >> $GITHUB_OUTPUT
 
-  # A job to identify and run the tests
+  # A job to run the tests
   pytest:
     needs: identify-tests
     runs-on: self-hosted

.github/workflows/test_changes.yml

@@ -38,7 +38,7 @@ jobs:
         echo $test_files
         echo "tests=$test_files" >> $GITHUB_OUTPUT
 
-  # A job to list the tests to be run
+  # A job to run the tests
   pytest:
     needs: identify-tests
     runs-on: self-hosted

docs/status.md

@@ -36,7 +36,7 @@ print(pipeline_info['status'])
 report.markdown() # Returns a string containing the markdown
 ```
 
-You can also use the command-line tool as so. Option `-t` is for the team id, option `-d` allows to print only one of the sub parts of the description among : `general`, `exclusions`, `preprocessing`, `analysis`, and `categorized_for_analysis`.
+You can also use the command-line tool as so.
 
 ```bash
 python narps_open/utils/status -h

narps_open/pipelines/__init__.py

@@ -62,7 +62,7 @@
     'O6R6': None,
     'P5F3': None,
     'Q58J': None,
-    'Q6O0': None,
+    'Q6O0': 'PipelineTeamQ6O0',
     'R42Q': None,
     'R5K7': None,
     'R7D1': None,