Summary

Fix formatting errors in hostingsvcs module according to PEP8

Review Request #6634 — Created Nov. 23, 2014 and submitted Nov. 30, 2014, 2:52 p.m.

Information

Owner

justy777

Repository

Review Board

Branch

master

Bugs

Depends On

Commit

44bd8d6...

Reviewers

Groups

reviewboard, students

People

Description

Fixed the following format errors:
hostingsvcs/codebasehq.py:73:9: E265 block comment should start with '# '
hostingsvcs/codebasehq.py:77:9: E265 block comment should start with '# '
hostingsvcs/codebasehq.py:83:9: E265 block comment should start with '# '
hostingsvcs/codebasehq.py:89:9: E265 block comment should start with '# '
hostingsvcs/github.py:280:5: E303 too many blank lines (2)
hostingsvcs/github.py:1080:80: E501 line too long (80 > 79 characters)
hostingsvcs/gitlab.py:129:30: E203 whitespace before ':'
hostingsvcs/gitlab.py:398:1: W391 blank line at end of file
hostingsvcs/tests.py:1393:80: E501 line too long (99 > 79 characters)
hostingsvcs/tests.py:1423:80: E501 line too long (99 > 79 characters)

Testing Done

Ran unit tests for hostingsvcs module, and ran pep8 again.

Issues

Description	From	Last Updated
Maybe just change the whole line, since it's a test diff. "Add a rule for clearing floats," maybe. Same below.	chipx86	Nov. 24, 2014, 3:28 p.m.
Same.	chipx86	Nov. 24, 2014, 3:28 p.m.

Tool: Pyflakes
Processed Files:
    reviewboard/hostingsvcs/codebasehq.py
    reviewboard/hostingsvcs/github.py
    reviewboard/hostingsvcs/gitlab.py
    reviewboard/hostingsvcs/tests.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/hostingsvcs/codebasehq.py
    reviewboard/hostingsvcs/github.py
    reviewboard/hostingsvcs/gitlab.py
    reviewboard/hostingsvcs/tests.py

Commit:

662d1e491d0e1fa5fefd643914b6dcdeb6404c00

0279835cbf4b8aa1ee36c819b45c66fdf0ec720e

Diff:

Revision 2 (+11 -13)

Show changes

	reviewboard/hostingsvcs/codebasehq.py
	reviewboard/hostingsvcs/github.py
	reviewboard/hostingsvcs/gitlab.py
	reviewboard/hostingsvcs/tests.py

Tool: Pyflakes
Processed Files:
    reviewboard/hostingsvcs/codebasehq.py
    reviewboard/hostingsvcs/github.py
    reviewboard/hostingsvcs/gitlab.py
    reviewboard/hostingsvcs/tests.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/hostingsvcs/codebasehq.py
    reviewboard/hostingsvcs/github.py
    reviewboard/hostingsvcs/gitlab.py
    reviewboard/hostingsvcs/tests.py

reviewboard/hostingsvcs/tests.py (Diff revision 2)

The issue has been resolved. Show all issues

Maybe just change the whole line, since it's a test diff. "Add a rule for clearing floats," maybe.

Same below.

reviewboard/hostingsvcs/tests.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Same.
```

Commit:

0279835cbf4b8aa1ee36c819b45c66fdf0ec720e

44bd8d68060331d3a9177e455744bdb47862a282

Diff:

Revision 3 (+11 -13)

Show changes

	reviewboard/hostingsvcs/codebasehq.py
	reviewboard/hostingsvcs/github.py
	reviewboard/hostingsvcs/gitlab.py
	reviewboard/hostingsvcs/tests.py

Tool: Pyflakes
Processed Files:
    reviewboard/hostingsvcs/codebasehq.py
    reviewboard/hostingsvcs/github.py
    reviewboard/hostingsvcs/gitlab.py
    reviewboard/hostingsvcs/tests.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/hostingsvcs/codebasehq.py
    reviewboard/hostingsvcs/github.py
    reviewboard/hostingsvcs/gitlab.py
    reviewboard/hostingsvcs/tests.py

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (a62bab7)