pre-commit/flake8
1
0
Fork 0
mirror of https://github.com/PyCQA/flake8.git synced 2026-04-02 19:56:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Only skip a file if # flake8: noqa is on a line by itself

Browse source
This commit is contained in:
Anthony Sottile 2017-12-20 12:43:45 -08:00
parent eb6228b660
commit e73055432c
2 changed files with 13 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/flake8/processor.py
View file

@ -106,7 +106,7 @@ class FileProcessor(object):
@property
def file_tokens(self):
"""The complete set of tokens for a file.
"""Return the complete set of tokens for a file.
Accessing this attribute *may* raise an InvalidSyntax exception.
@ -334,7 +334,7 @@ class FileProcessor(object):
def should_ignore_file(self):
# type: () -> bool
"""Check if ``# flake8: noqa`` is in the file to be ignored.
"""Check if ``flake8: noqa`` is in the file to be ignored.
:returns:
True if a line matches :attr:`defaults.NOQA_FILE`,
@ -342,8 +342,16 @@ class FileProcessor(object):
:rtype:
bool
"""
ignore_file = defaults.NOQA_FILE.search
return any(ignore_file(line) for line in self.lines)
if any(defaults.NOQA_FILE.match(line) for line in self.lines):
return True
elif any(defaults.NOQA_FILE.search(line) for line in self.lines):
LOG.warning(
'Detected `flake8: noqa` on line with code. To ignore an '
'error on a line use `noqa` instead.',
)
return False
else:
return False
def strip_utf_bom(self):
# type: () -> NoneType