Skip to content

Commit

Permalink
Remove pylint check
Browse files Browse the repository at this point in the history
We use flake8 which covers the same purpose.
  • Loading branch information
elfosardo committed May 12, 2020
1 parent a1dc461 commit a672703
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 337 deletions.
6 changes: 0 additions & 6 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,4 @@
language: python
addons:
apt:
packages:
- bc
- findutils
sudo: false
dist: bionic
python:
Expand All @@ -13,7 +8,6 @@ python:
env:
- TOXENV=$(echo py$TRAVIS_PYTHON_VERSION | tr -d .)
- TOXENV=pep8
- TOXENV=pylint
- TOXENV=cover
- TOXENV=docs
install:
Expand Down
1 change: 0 additions & 1 deletion test-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,4 @@ python-subunit # Apache-2.0/BSD
stestr>=2.0.0 # Apache-2.0
flake8-import-order>=0.13
testtools>=0.9.34 # MIT
pylint>=1.4.2 # GPL
Babel!=2.4.0,>=2.3.4 # BSD
43 changes: 0 additions & 43 deletions tools/check-pylint-score.sh

This file was deleted.

279 changes: 0 additions & 279 deletions tools/pylintrc

This file was deleted.

9 changes: 1 addition & 8 deletions tox.ini
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[tox]
minversion = 3.1.0
envlist = py36,py37,py38,pep8,pylint,cover,docs
envlist = py36,py37,py38,pep8,cover,docs
skipsdist = True
ignore_basepython_conflict=true

Expand Down Expand Up @@ -41,13 +41,6 @@ commands =
coverage html -d cover --omit='*tests*'
coverage xml -o cover/coverage.xml --omit='*tests*'

[testenv:pylint]
whitelist_externals =
/bin/sh
/bin/bash
/usr/bin/bash
commands = bash -c "{toxinidir}/tools/check-pylint-score.sh 8.91 -j $(nproc) --rcfile {toxinidir}/tools/pylintrc $(find {toxinidir}/hardware -name '*.py'|grep -v '^{toxinidir}/hardware/tests')"

[testenv:docs]
deps = -r{toxinidir}/doc/requirements.txt
commands = sphinx-build -b html -W doc/source doc/build/html
Expand Down

0 comments on commit a672703

Please sign in to comment.