• 
      

    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