PEP8 1.3 compliance
@jeff
Submitted by Jeff F.T. Assigned to Jeff F.T. @jeff
Description
Our code is compliant to pep8 1.0.1, but will cause warnings with the 1.3.x version of PEP8 (which will be in most distro releases this fall).
The most frequent errors are E128 and E127. It is about indentation of code that is split on several lines. Some statistics:
23 E111 indentation is not a multiple of four
12 E121 continuation line indentation is not a multiple of four
10 E123 closing bracket does not match indentation of opening bracket's line
9 E124 closing bracket does not match visual indentation
29 E125 continuation line does not distinguish itself from next logical line
41 E126 continuation line over-indented for hanging indent
71 E127 continuation line over-indented for visual indent
558 E128 continuation line under-indented for visual indent
1 E211 whitespace before '('
2 E221 multiple spaces before operator
12 E225 missing whitespace around operator
3 E231 missing whitespace after ','
2 E261 at least two spaces before inline comment
2 E262 inline comment should start with '# '
1 E271 multiple spaces after keyword
2 E301 expected 1 blank line, found 0
9 E302 expected 2 blank lines, found 1
6 E303 too many blank lines (2)
577 E501 line too long (101 > 79 characters)
4 E502 the backslash is redundant between brackets
24 E711 comparison to None should be 'if cond is not None:'
Currently, dealing with 500 changes for the indentation of continued lines is quite invasive and should be done only once everything else is done (the pygi branch merged to master, etc).
For now, those running PEP8 need to ignore those errors locally.
Imported from https://bugzilla.gnome.org/show_bug.cgi?id=681922