Fixed a ton of PEP-8 issues in the docs.

Review Request #6639 — Created Nov. 24, 2014 and submitted

Information

Review Board
master
eb586c5...

Reviewers

Fixed the following PEP-8 issues:
docs/codebase/conf.py:6:80: E501 line too long (80 > 79 characters)
docs/codebase/conf.py:9:80: E501 line too long (82 > 79 characters)
docs/codebase/conf.py:17:11: E401 multiple imports on one line
docs/codebase/conf.py:37:80: E501 line too long (80 > 79 characters)
docs/codebase/conf.py:39:80: E501 line too long (83 > 79 characters)
docs/codebase/conf.py:83:80: E501 line too long (80 > 79 characters)
docs/codebase/conf.py:179:80: E501 line too long (81 > 79 characters)
docs/manual/conf.py:6:80: E501 line too long (80 > 79 characters)
docs/manual/conf.py:9:80: E501 line too long (82 > 79 characters)
docs/manual/conf.py:45:80: E501 line too long (80 > 79 characters)
docs/manual/conf.py:99:80: E501 line too long (80 > 79 characters)
docs/manual/conf.py:197:80: E501 line too long (81 > 79 characters)
docs/manual/_ext/webapidocs.py:383:24: E126 continuation line over-indented for hanging indent
docs/manual/_ext/webapidocs.py:660:80: E501 line too long (80 > 79 characters)
docs/manual/_ext/webapidocs.py:866:1: E302 expected 2 blank lines, found 1
docs/manual/_ext/webapidocs.py:894:1: E302 expected 2 blank lines, found 1
docs/releasenotes/conf.py:6:80: E501 line too long (80 > 79 characters)
docs/releasenotes/conf.py:9:80: E501 line too long (82 > 79 characters)
docs/releasenotes/conf.py:38:80: E501 line too long (80 > 79 characters)
docs/releasenotes/conf.py:85:80: E501 line too long (80 > 79 characters)
docs/releasenotes/conf.py:183:80: E501 line too long (81 > 79 characters)
docs/releasenotes/_ext/extralinks.py:28:30: E713 test for membership should be 'not in'

Built docs, and pep8.

Description From Last Updated

Swap these. Alphabtical order.

chipx86chipx86

Despite what the tool says, the lack of a space on these is correct. That's because it's not a comment, …

chipx86chipx86

Can you instead put the whole expression in parens, and remove the backslash? Then the expressions can align.

chipx86chipx86

Same with these.

chipx86chipx86

Same comments as above for this file.

chipx86chipx86
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        docs/releasenotes/_ext/extralinks.py
        docs/manual/conf.py
        docs/manual/_ext/webapidocs.py
        docs/codebase/conf.py
        docs/releasenotes/conf.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        docs/releasenotes/_ext/extralinks.py
        docs/manual/conf.py
        docs/manual/_ext/webapidocs.py
        docs/codebase/conf.py
        docs/releasenotes/conf.py
    
    
  2. 
      
chipx86
  1. 
      
  2. docs/codebase/conf.py (Diff revision 1)
     
     
     
    Show all issues

    Swap these. Alphabtical order.

  3. docs/codebase/conf.py (Diff revision 1)
     
     
    Show all issues

    Despite what the tool says, the lack of a space on these is correct. That's because it's not a comment, but rather a commented-out line.

  4. docs/manual/_ext/webapidocs.py (Diff revision 1)
     
     
     
    Show all issues

    Can you instead put the whole expression in parens, and remove the backslash? Then the expressions can align.

  5. docs/manual/conf.py (Diff revision 1)
     
     
    Show all issues

    Same with these.

  6. docs/releasenotes/conf.py (Diff revision 1)
     
     
    Show all issues

    Same comments as above for this file.

  7. 
      
justy777
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        docs/releasenotes/_ext/extralinks.py
        docs/manual/conf.py
        docs/manual/_ext/webapidocs.py
        docs/codebase/conf.py
        docs/releasenotes/conf.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        docs/releasenotes/_ext/extralinks.py
        docs/manual/conf.py
        docs/manual/_ext/webapidocs.py
        docs/codebase/conf.py
        docs/releasenotes/conf.py
    
    
  2. 
      
david
  1. Have you tested building the docs?

  2. 
      
justy777
brennie
  1. Ship It!
  2. 
      
chipx86
  1. Ship It!
  2. 
      
justy777
Review request changed
Status:
Completed
Change Summary:
Pushed to master (79a0526)