Fix formatting errors in scmtools module according to PEP8
Review Request #6636 — Created Nov. 24, 2014 and submitted
| Information | |
|---|---|
| justy777 | |
| Review Board | |
| master | |
| 5087460... | |
| Reviewers | |
| reviewboard, students | |
Fixed the following format errors:
scmtools/clearcase.py:86:12: E713 test for membership should be 'not in'
scmtools/clearcase.py:274:18: E713 test for membership should be 'not in'
scmtools/clearcase.py:418:1: E302 expected 2 blank lines, found 1
scmtools/cvs.py:343:17: E713 test for membership should be 'not in'
scmtools/svn/init.py:41:12: E221 multiple spaces before operator
scmtools/svn/init.py:42:16: E221 multiple spaces before operator
scmtools/svn/init.py:43:15: E221 multiple spaces before operator
scmtools/svn/base.py:25:18: E241 multiple spaces after ':'
scmtools/svn/base.py:26:16: E241 multiple spaces after ':'
scmtools/svn/base.py:27:20: E241 multiple spaces after ':'
scmtools/svn/base.py:28:19: E241 multiple spaces after ':'
scmtools/svn/base.py:29:14: E241 multiple spaces after ':'
scmtools/svn/base.py:30:18: E241 multiple spaces after ':'
scmtools/svn/base.py:33:25: E241 multiple spaces after ':'
scmtools/svn/base.py:34:27: E241 multiple spaces after ':'
scmtools/svn/base.py:36:15: E241 multiple spaces after ':'
scmtools/svn/base.py:37:15: E241 multiple spaces after ':'
scmtools/tests.py:557:80: E501 line too long (88 > 79 characters)
scmtools/tests.py:606:80: E501 line too long (88 > 79 characters)
scmtools/tests.py:619:80: E501 line too long (88 > 79 characters)
scmtools/tests.py:633:80: E501 line too long (88 > 79 characters)
scmtools/tests.py:653:80: E501 line too long (88 > 79 characters)
scmtools/tests.py:678:80: E501 line too long (87 > 79 characters)
scmtools/tests.py:2691:80: E501 line too long (81 > 79 characters)
scmtools/tests.py:2829:80: E501 line too long (81 > 79 characters)
scmtools/tests.py:2921:80: E501 line too long (80 > 79 characters)
Ran unit tests for scmtools module, and ran pep8 again.
-
-
reviewboard/scmtools/managers.py (Diff revision 1) I prefer the old way, even if pep8 is upset about it.
-
reviewboard/scmtools/tests.py (Diff revision 1) We're matching what diffs provide, so this cannot be changed.
Same below.
-
Description: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Commit: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 2 (+35 -25) |
-
Tool: Pyflakes Processed Files: reviewboard/scmtools/cvs.py reviewboard/scmtools/clearcase.py reviewboard/scmtools/svn/__init__.py reviewboard/scmtools/svn/base.py reviewboard/scmtools/tests.py Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/cvs.py reviewboard/scmtools/clearcase.py reviewboard/scmtools/svn/__init__.py reviewboard/scmtools/svn/base.py reviewboard/scmtools/tests.py
Tool: Pyflakes Processed Files: reviewboard/scmtools/clearcase.py reviewboard/scmtools/svn/base.py reviewboard/scmtools/managers.py reviewboard/scmtools/tests.py reviewboard/scmtools/cvs.py reviewboard/scmtools/svn/__init__.py Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/clearcase.py reviewboard/scmtools/svn/base.py reviewboard/scmtools/managers.py reviewboard/scmtools/tests.py reviewboard/scmtools/cvs.py reviewboard/scmtools/svn/__init__.py