Remove E999 and replace them with todos.

This commit is contained in:
Dave Halter
2017-07-08 12:07:00 +02:00
parent 1e2abec7f2
commit 3943618fe2
4 changed files with 4 additions and 6 deletions

View File

@@ -88,10 +88,10 @@ if length > options.max_line_length:
if os.path.exists(os.path.join(path, PEP8_BIN)): if os.path.exists(os.path.join(path, PEP8_BIN)):
cmd = ([os.path.join(path, PEP8_BIN)] + cmd = ([os.path.join(path, PEP8_BIN)] +
self._pep8_options(targetfile)) self._pep8_options(targetfile))
#: E999+1 # TODO Tabs in docstrings shouldn't be there, use \t.
''' '''
multiline string with tab in it''' multiline string with tab in it'''
#: E999+1 # Same here.
'''multiline string '''multiline string
with tabs with tabs
and spaces and spaces

View File

@@ -94,7 +94,7 @@ c = (a+b) * (a-b)
norman = True+False norman = True+False
x = x*2 - 1 x = x*2 - 1
x = x/2 - 1 x = x/2 - 1
#: E999 # TODO whitespace should be the other way around according to pep8.
x = x / 2-1 x = x / 2-1
hypot2 = x*x + y*y hypot2 = x*x + y*y

View File

@@ -19,7 +19,7 @@ parser.add_argument('--long-option',
parser.add_argument('--long-option', parser.add_argument('--long-option',
default= default=
"/rather/long/filesystem") "/rather/long/filesystem")
#: E999+1:44 # TODO this looks so stupid.
parser.add_argument('--long-option', default parser.add_argument('--long-option', default
="/rather/long/filesystem/path/here/blah/blah/blah") ="/rather/long/filesystem/path/here/blah/blah/blah")
#: E251+2:7 E251+2:9 #: E251+2:7 E251+2:9

View File

@@ -44,8 +44,6 @@ def collect_errors(code):
code, _, add_line = code.partition('+') code, _, add_line = code.partition('+')
l = line_nr + 1 + int(add_line or 0) l = line_nr + 1 + int(add_line or 0)
if code == 'E999': continue # TODO delete
yield WantedIssue(code[1:], l, column) yield WantedIssue(code[1:], l, column)