Setup Python linters in tox 65/103265/1
authorBartek Grzybowski <b.grzybowski@partner.samsung.com>
Fri, 6 Mar 2020 13:20:07 +0000 (14:20 +0100)
committerBartek Grzybowski <b.grzybowski@partner.samsung.com>
Fri, 6 Mar 2020 13:20:07 +0000 (14:20 +0100)
This adds Python linting setup that'll be triggered
by relevant CI job for any uploaded python script review.

Change-Id: I957156e281e97a05651e806f6418a7c450e20b43
Issue-ID: INT-1427
Signed-off-by: Bartek Grzybowski <b.grzybowski@partner.samsung.com>
.coafile
tox.ini

index a22c086..06c5301 100644 (file)
--- a/.coafile
+++ b/.coafile
@@ -16,3 +16,15 @@ ignore =
   test/mocks/pnfsimulator/pnfsimulator/src/test/resources/org/onap/pnfsimulator/simulator/invalidJsonStructureEvent.json,
   # This one is empty
   test/mocks/datafilecollector-testharness/simulator-group/consul/consul_config.json
+
+[py]
+bears = PyLintBear
+pylint_disable = all
+pylint_enable =
+  bad-indentation, trailing-whitespace, unused-wildcard-import, unused-import,
+  unnecessary-semicolon, unnecessary-semicolon, undefined-variable,
+  syntax-error, unused-variable, using-constant-test,unused-argument,
+  len-as-condition, trailing-newlines, missing-final-newline, reimported,
+  too-many-function-args, singleton-comparison
+ignore =
+  .tox/**
diff --git a/tox.ini b/tox.ini
index 0d17113..68745ff 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -1,6 +1,6 @@
 [tox]
 minversion = 3.2.0
-envlist = json,yaml
+envlist = json,yaml,py
 skipsdist = true
 requires = pip >= 8
 
@@ -27,3 +27,9 @@ commands =
 # '\ ' at the end of command is needed for a case where above command returns empty list (it adds empty file
 # parameter to '--files' opt
     /bin/bash -c "coala --non-interactive --disable-caching --no-autoapply-warn yaml --files $(</tmp/.coalist_yaml) \ "
+
+[testenv:py]
+commands_pre =
+    /bin/sh -c "git --no-pager diff HEAD HEAD^ --name-only '*.py' > /tmp/.coalist_py"
+commands =
+    /bin/bash -c "coala --non-interactive --disable-caching --no-autoapply-warn py --files $(</tmp/.coalist_py) \ "