add a gating errors-only Pylint check to CI

As of the previous commit, the Docker image on which Pylint checks are run in CI
has all Python module dependencies available, so import-error warnings are no
longer an issue. This change adds a build-failing more relaxed Pylint check to
the existing (non-build-failing) one, that only checks for conclusive errors. It
should no longer be possible to introduce broken Python code into Graphviz –
even when it is uncalled – without breaking CI.

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 2ed28c55912156e9af2382ae5f50d437048bda57..2b7ccf842104dd1f50b743bd65f918a827b4e8c5 100644 (file)
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1067,6 +1067,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`
   artifacts:
     when: on_success
     expire_in: 1 week