make final Pylint step fail for any non-FIXME warning/error

The gating Pylint check now fails if any warning or error is detected, excluding
FIXME comments. This should hopefully lead to improved code quality in Python
introduced into the code base in future.

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index f4596a1041c7fd071017837d4892a154d84d0389..2f50a49588273c79b9c30c9dff8c7e9019ef8db8 100644 (file)
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1141,7 +1141,7 @@ lint_pylint:
     - echo "$CI_JOB_NAME-warnings `egrep -c '\<[CRWEF][[:digit:]]{4}\>' $logfile`" > metrics.txt
     - rm $logfile
     - cat metrics.txt
-    - python3 -m pylint --errors-only --rcfile=.pylintrc `find . -name '*.py' | xargs`
+    - python3 -m pylint --rcfile=.pylintrc --disable=fixme `find . -name '*.py' | xargs`
   artifacts:
     when: on_success
     expire_in: 1 week