Fix formatting errors in attachments module according to PEP8

Review Request #6627 — Created Nov. 23, 2014 and submitted

Information

Review Board
master
44ce6b8...

Reviewers

Fixed the following errors:
attachments/tests.py:215:80: E501 line too long (81 > 79 characters)

Ran unit tests for attachments module, and ran pep8 again.

Description From Last Updated

Can we put f.read() on its own line? Generally when we use this form of indentation, we tend to do …

chipx86chipx86
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/attachments/tests.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        reviewboard/attachments/tests.py
    
    
  2. 
      
chipx86
  1. 
      
  2. reviewboard/attachments/tests.py (Diff revision 1)
     
     

    Can we put f.read() on its own line? Generally when we use this form of indentation, we tend to do one per line.

  3. 
      
justy777
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/attachments/tests.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        reviewboard/attachments/tests.py
    
    
  2. 
      
chipx86
  1. 
      
  2. 
      
justy777
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (892c17b)
Loading...