Clean up function name and docstring issues throughout the codebase.

Review Request #6062 — Created July 4, 2014 and submitted

Information

Review Board
release-2.0.x
d77a98d...

Reviewers

This updates almost all the camelCase function names found in codebase
to use our standard naming convention. In some places, it also updates
docstrings to be more useful and clear. This will help contributors to
see good (or at least better) examples of how their tests and functions
should look.

This does not update any assert* function names, nor does it update
anything in the webapi codebase (since this is handled by another
change).

Unit tests pass.

Description From Last Updated

Col: 80 E501 line too long (81 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (81 > 79 characters)

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/admin/widgets.py
        reviewboard/reviews/tests.py
        reviewboard/diffviewer/tests.py
        reviewboard/changedescs/tests.py
        reviewboard/datagrids/tests.py
        reviewboard/webapi/tests/base.py
        reviewboard/reviews/management/commands/fill-database.py
        reviewboard/admin/tests.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/admin/widgets.py
        reviewboard/reviews/tests.py
        reviewboard/diffviewer/tests.py
        reviewboard/changedescs/tests.py
        reviewboard/datagrids/tests.py
        reviewboard/webapi/tests/base.py
        reviewboard/reviews/management/commands/fill-database.py
        reviewboard/admin/tests.py
    
    
  2. reviewboard/datagrids/tests.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (81 > 79 characters)
    
  3. reviewboard/datagrids/tests.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (81 > 79 characters)
    
  4. 
      
chipx86
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/admin/widgets.py
        reviewboard/reviews/tests.py
        reviewboard/diffviewer/tests.py
        reviewboard/changedescs/tests.py
        reviewboard/datagrids/tests.py
        reviewboard/webapi/tests/base.py
        reviewboard/reviews/management/commands/fill-database.py
        reviewboard/admin/tests.py
    
    
  2. 
      
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/admin/widgets.py
        reviewboard/reviews/tests.py
        reviewboard/diffviewer/tests.py
        reviewboard/changedescs/tests.py
        reviewboard/datagrids/tests.py
        reviewboard/webapi/tests/base.py
        reviewboard/reviews/management/commands/fill-database.py
        reviewboard/admin/tests.py
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (2d83766)
Loading...