From b011ebb0e7e8ed7f4591e9d37ddc62a45b1cd927 Mon Sep 17 00:00:00 2001 From: Tim Burke Date: Thu, 4 Feb 2016 15:33:16 -0800 Subject: [PATCH] Run all flake8 checks Previously, only H106 was being run. Depends-On: I04b862ced4659dfeb3c515edb3b8fed36b5b086d Change-Id: I76b5bbfee016ea25904a558c64a69b796d135f2d --- hacking/checks/comments.py | 4 ++-- hacking/checks/vim_check.py | 2 +- requirements.txt | 2 +- tox.ini | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/hacking/checks/comments.py b/hacking/checks/comments.py index e66e9cb..96eaa8f 100644 --- a/hacking/checks/comments.py +++ b/hacking/checks/comments.py @@ -162,8 +162,8 @@ under the License.""" if stripped_apache2 in content: return True else: - print ("!=:\n'%s' !=\n'%s'" % - (content, stripped_apache2)) + print("!=:\n'%s' !=\n'%s'" % + (content, stripped_apache2)) return False diff --git a/hacking/checks/vim_check.py b/hacking/checks/vim_check.py index 17c501a..a9c695a 100644 --- a/hacking/checks/vim_check.py +++ b/hacking/checks/vim_check.py @@ -35,5 +35,5 @@ def no_vim_headers(physical_line, line_number, lines): Okay: # vim, ze nemluvis cesky """ if ((line_number <= 5 or line_number > len(lines) - 5) and - vim_header_re.match(physical_line)): + vim_header_re.match(physical_line)): return 0, "H106: Don't put vim configuration in source files" diff --git a/requirements.txt b/requirements.txt index c98199b..7146fce 100644 --- a/requirements.txt +++ b/requirements.txt @@ -5,7 +5,7 @@ pbr>=1.6 # Apache-2.0 pep8==1.5.7 # MIT pyflakes==0.8.1 # MIT -flake8<2.6.0,>2.4.1 # MIT +flake8<2.6.0,>=2.5.4 # MIT mccabe==0.2.1 # MIT License six>=1.9.0 # MIT diff --git a/tox.ini b/tox.ini index f066825..9a54fda 100644 --- a/tox.ini +++ b/tox.ini @@ -37,7 +37,7 @@ commands = python setup.py build_sphinx [flake8] exclude = .venv,.tox,dist,doc,*.egg,build show-source = true -select = H106 +enable-extensions = H106 [hacking] local-check = hacking.tests.test_local.check