Use pylint for CRLF checking.
This commit is contained in:
parent
2b058b36ea
commit
be21296b38
28
scripts/pylint_checkers/crlf.py
Normal file
28
scripts/pylint_checkers/crlf.py
Normal file
@ -0,0 +1,28 @@
|
|||||||
|
"""Checker for CRLF in files."""
|
||||||
|
|
||||||
|
from pylint.interfaces import IRawChecker
|
||||||
|
from pylint.checkers import BaseChecker
|
||||||
|
|
||||||
|
|
||||||
|
class CrlfChecker(BaseChecker):
|
||||||
|
|
||||||
|
"""Check for CRLF in files."""
|
||||||
|
|
||||||
|
__implements__ = IRawChecker
|
||||||
|
|
||||||
|
name = 'crlf'
|
||||||
|
msgs = {'W9001': ('Uses CRLFs', 'crlf', None)}
|
||||||
|
options = ()
|
||||||
|
priority = -1
|
||||||
|
|
||||||
|
def process_module(self, node):
|
||||||
|
"""Process the module."""
|
||||||
|
for (lineno, line) in enumerate(node.file_stream):
|
||||||
|
if b'\r\n' in line:
|
||||||
|
self.add_message('crlf', line=lineno)
|
||||||
|
return
|
||||||
|
|
||||||
|
|
||||||
|
def register(linter):
|
||||||
|
"""Register the checker."""
|
||||||
|
linter.register_checker(CrlfChecker(linter))
|
@ -68,7 +68,8 @@ options = {
|
|||||||
'other': {
|
'other': {
|
||||||
'pylint': ['--output-format=colorized', '--reports=no',
|
'pylint': ['--output-format=colorized', '--reports=no',
|
||||||
'--rcfile=.pylintrc',
|
'--rcfile=.pylintrc',
|
||||||
'--load-plugins=pylint_checkers.config'],
|
'--load-plugins=pylint_checkers.config,'
|
||||||
|
'pylint_checkers.crlf'],
|
||||||
'flake8': ['--config=.flake8'],
|
'flake8': ['--config=.flake8'],
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
@ -198,10 +199,7 @@ def _check_file(fn):
|
|||||||
ok = True
|
ok = True
|
||||||
with open(fn, 'rb') as f:
|
with open(fn, 'rb') as f:
|
||||||
for line in f:
|
for line in f:
|
||||||
if b'\r\n' in line:
|
if any(line.decode('UTF-8').startswith(c * 7) for c in "<>=|"):
|
||||||
print("Found CRLF in {}".format(fn))
|
|
||||||
ok = False
|
|
||||||
elif any(line.decode('UTF-8').startswith(c * 7) for c in "<>=|"):
|
|
||||||
print("Found conflict marker in {}".format(fn))
|
print("Found conflict marker in {}".format(fn))
|
||||||
ok = False
|
ok = False
|
||||||
elif b'set_trace()' in line and not (
|
elif b'set_trace()' in line and not (
|
||||||
|
Loading…
Reference in New Issue
Block a user