summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Stapleton Cordasco <graffatcolmingov@gmail.com>2017-07-28 19:31:32 -0500
committerIan Stapleton Cordasco <graffatcolmingov@gmail.com>2017-07-28 19:31:32 -0500
commitec996ffd85cbe6157b7cf480e7c84b36d2153f4b (patch)
tree636e2002ea04d201e02f33edbd2af1dfac57051e
parenta4f2135c774c31a7988a7cadbf793fe1c9271284 (diff)
downloadflake8-ec996ffd85cbe6157b7cf480e7c84b36d2153f4b.tar.gz
Allow spaces in # noqa lists
To match our new configuration file format and its allowance for spaces in the list Closes #356
-rw-r--r--src/flake8/defaults.py2
-rw-r--r--tests/unit/test_utils.py1
-rw-r--r--tests/unit/test_violation.py1
3 files changed, 3 insertions, 1 deletions
diff --git a/src/flake8/defaults.py b/src/flake8/defaults.py
index e8c6bfb..55cb48a 100644
--- a/src/flake8/defaults.py
+++ b/src/flake8/defaults.py
@@ -46,7 +46,7 @@ NOQA_INLINE_REGEXP = re.compile(
# We do not care about the ``: `` that follows ``noqa``
# We do not care about the casing of ``noqa``
# We want a comma-separated list of errors
- '# noqa(?:: (?P<codes>([A-Z][0-9]+,?)+))?',
+ '# noqa(?:: (?P<codes>([A-Z][0-9]+(?:[,\s]+)?)+))?',
re.IGNORECASE
)
diff --git a/tests/unit/test_utils.py b/tests/unit/test_utils.py
index e50dade..9f3976f 100644
--- a/tests/unit/test_utils.py
+++ b/tests/unit/test_utils.py
@@ -19,6 +19,7 @@ RELATIVE_PATHS = ["flake8", "pep8", "pyflakes", "mccabe"]
("E123,W234,E206,", ["E123", "W234", "E206"]),
("E123,W234,E206, ,\n", ["E123", "W234", "E206"]),
("E123,W234,,E206,,", ["E123", "W234", "E206"]),
+ ("E123, W234,, E206,,", ["E123", "W234", "E206"]),
("E123,,W234,,E206,,", ["E123", "W234", "E206"]),
(["E123", "W234", "E206"], ["E123", "W234", "E206"]),
(["E123", "\n\tW234", "\n E206"], ["E123", "W234", "E206"]),
diff --git a/tests/unit/test_violation.py b/tests/unit/test_violation.py
index fa75278..c4b56b1 100644
--- a/tests/unit/test_violation.py
+++ b/tests/unit/test_violation.py
@@ -11,6 +11,7 @@ from flake8 import style_guide
('E121', 'a = 1 # noqa: E111,W123,F821', False),
('E111', 'a = 1 # noqa: E111,W123,F821', True),
('W123', 'a = 1 # noqa: E111,W123,F821', True),
+ ('W123', 'a = 1 # noqa: E111, W123,F821', True),
('E111', 'a = 1 # noqa: E11,W123,F821', True),
('E111', 'a = 1 # noqa, analysis:ignore', True),
('E111', 'a = 1 # noqa analysis:ignore', True),