Fix linters in CI (#4399)
* fix flake8 * unify mountpoint * remove javascript linting as we already use codacy and LGTM for that * add python.d.plugin.in
Paweł Krupa committed
Oct 16, 2018 at 14:34 UTC
4f84553396fcd3cba3ef2f38726edf81cae1443c
2 files changed
+6
-9
.travis.yml
+3
-6
@@ -21,16 +21,13 @@ jobs:
21
- stage: "lint"
22
name: "python"
23
install: skip
24
- script: docker run -ti --rm -v $(pwd):/apps alpine/flake8:3.5.0 --max-line-length=120 --exclude=third_party,urllib3,pyyaml3,pyyaml2 python.d/ plugins.d/python.d.plugin
25
- - name: "javascript"
26
- install: skip
27
- script: docker run -it --rm -v $(pwd)/web:/code eeacms/jslint --color /code/web/*.js /code/plugins.d/node.d.plugin/*.js /code/node.d/*.js /code/node.d/node_modules/netdata.js
24
+ script: docker run -ti --rm -v $(pwd):/code alpine/flake8:3.5.0 --max-line-length=120 --exclude=third_party,urllib3,pyyaml3,pyyaml2 /code/collectors/python.d.plugin/python.d.plugin.in /code/
25
- name: "css"
26
install: skip
30
- script: docker run --rm -v $(pwd)/web:/code eeacms/csslint sh -c "csslint /code/*.css" || echo "OK"
27
+ script: docker run --rm -v $(pwd):/code eeacms/csslint sh -c "csslint /code/web/gui/*.css" || echo "OK"
28
- name: "bash"
29
install: skip
33
- script: docker run --rm --volume "$(pwd)":/project:ro --entrypoint sh koalaman/shellcheck-alpine:v0.5.0 -c 'for file in $(find /project/ -type f -name "*.sh" ! -path "*/makeself/*" ); do if ! shellcheck --format=gcc $file; then export FAILED=true; fi; done; if [ "$FAILED" != "" ]; then exit 1; fi'
30
+ script: docker run --rm --volume "$(pwd)":/code:ro --entrypoint sh koalaman/shellcheck-alpine:v0.5.0 -c 'for file in $(find /code/ -type f -name "*.sh" ! -path "*/makeself/*" ); do if ! shellcheck --format=gcc $file; then export FAILED=true; fi; done; if [ "$FAILED" != "" ]; then exit 1; fi'
31
- stage: "test"
32
name: "C"
33
install: sudo apt-get install -y libcap2-bin zlib1g-dev uuid-dev fakeroot libipmimonitoring-dev libmnl-dev libnetfilter-acct-dev
collectors/python.d.plugin/python.d.plugin.in
+3
-3
@@ -38,9 +38,9 @@ PYTHON_MODULES_DIR = os.path.join(PLUGINS_DIR, 'python_modules')
38
39
sys.path.append(PYTHON_MODULES_DIR)
40
41
-from bases.loaders import ModuleAndConfigLoader
42
-from bases.loggers import PythonDLogger
43
-from bases.collection import setdefault_values, run_and_exit
41
+from bases.loaders import ModuleAndConfigLoader # noqa: E402
42
+from bases.loggers import PythonDLogger # noqa: E402
43
+from bases.collection import setdefault_values, run_and_exit # noqa: E402
44
45
try:
46
from collections import OrderedDict