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

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

justy777
Review Board
master
eb586c5...
reviewboard, students

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)
     
     
     

    Swap these. Alphabtical order.

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

    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)
     
     
     

    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)
     
     

    Same with these.

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

    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: Closed (submitted)

Change Summary:

Pushed to master (79a0526)
Loading...