Fix some PEP-257 issues in reviewboard/accounts/.

Review Request #6929 — Created Feb. 10, 2015 and submitted — Latest diff uploaded

Information

Review Board
master
f5fc853...

Reviewers

If we're eventually going to integrate the PEP-257 tool into Review Bot (which
I think we should), we should get our docstring house in order first. This
change goes through reviewboard/accounts and fixes most of the issues that
aren't just a missing docstring.

The one "error" that I chose to disable for this is D203, which wants a blank
line before class docstrings. I didn't see any justification for this in the
actual PEP-257 document, and I think it's ugly. There are also a couple cases
that I chose to ignore where our test docstrings which wrap didn't follow the
usual rules. There are a ton of things that have missing docstrings, but those
will require more work to clean up.

  • Ran unit tests
  • Ran pep257 --ignore=D203 reviewboard/accounts
    Loading...