Merge pull request pre-commit#373 from pre-commit/not_using_pylint

asottile · asottile · commit 6654fee5f9c4 · 2016-05-20T16:31:19.000-07:00
We're not using pylint
diff --git a/pre_commit/five.py b/pre_commit/five.py
@@ -1,6 +1,5 @@
 from __future__ import unicode_literals
 
-# pylint:disable=invalid-name
 PY2 = str is bytes
 PY3 = str is not bytes
 
diff --git a/pylintrc b/pylintrc
diff --git a/requirements-dev.txt b/requirements-dev.txt
@@ -1,10 +1,8 @@
 -e .
 
-astroid<1.3.3
 coverage
 flake8
 mock
-pylint<1.4
 pytest
 
 # setuptools breaks pypy3 with extraneous output
diff --git a/tests/commands/run_test.py b/tests/commands/run_test.py
@@ -457,9 +457,9 @@ def test_local_hook_for_stages(
     config = OrderedDict((
         ('repo', 'local'),
         ('hooks', (OrderedDict((
-            ('id', 'pylint'),
+            ('id', 'flake8'),
             ('name', 'hook 1'),
-            ('entry', 'python -m pylint.__main__'),
+            ('entry', 'python -m flake8.__main__'),
             ('language', 'system'),
             ('files', r'\.py$'),
             ('stages', stage_for_first_hook)
@@ -491,9 +491,9 @@ def test_local_hook_passes(repo_with_passing_hook, mock_out_store_directory):
     config = OrderedDict((
         ('repo', 'local'),
         ('hooks', (OrderedDict((
-            ('id', 'pylint'),
-            ('name', 'PyLint'),
-            ('entry', 'python -m pylint.__main__'),
+            ('id', 'flake8'),
+            ('name', 'flake8'),
+            ('entry', 'python -m flake8.__main__'),
             ('language', 'system'),
             ('files', r'\.py$'),
         )), OrderedDict((
diff --git a/tox.ini b/tox.ini
@@ -11,7 +11,6 @@ commands =
     coverage run -m pytest {posargs:tests}
     # TODO: when dropping py26, change to 100
     coverage report --show-missing --fail-under 99
-#    pylint {[tox]project} testing tests setup.py
     pre-commit run --all-files
 
 [testenv:venv]
