]> granicus.if.org Git - graphviz/commitdiff
CI: remove metrics collection in Pylint job
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 5 Mar 2022 21:14:59 +0000 (13:14 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 6 Mar 2022 16:44:31 +0000 (08:44 -0800)
The Pylint job was running two checks:

  1. Everything, with warning count piped to metrics.txt
  2. Everything except the check for FIXMEs, hard failing if there is any
     warning

The latter passes. That is, all Graphviz Python code is warning-free with the
exception of FIXMEs. Meanwhile FIXMEs are frequently _intentionally_ introduced
in test code, indicating a known open bug for which a test case is being added.
So metric collection in step 1 above is not relevant; we expect this number
often to go up. This commit removes the collection of this unused metric to
decrease CI complexity.

.gitlab-ci.yml

index 2ee2ee78295d5e23dab8aa5c967be70e8b638dff..d9eb95943f84a0c9604cb3aa8d364b33e5dd2aa8 100644 (file)
@@ -1502,14 +1502,6 @@ lint_pylint:
   needs:
     - docker_build_ubuntu-21.10
   script:
-    - logfile=`mktemp`
-    - python3 -m pylint --rcfile=.pylintrc `find . -name '*.py' | xargs` --exit-zero |& tee $logfile
-    - echo "$CI_JOB_NAME-warnings `egrep -c '\<[CRWEF][[:digit:]]{4}\>' $logfile`" | tee metrics.txt
     - python3 -m pylint --rcfile=.pylintrc --disable=fixme `find . -name '*.py' | xargs`
-  artifacts:
-    when: on_success
-    expire_in: 1 week
-    reports:
-        metrics: metrics.txt
   except:
     - tags