• 
      

    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)
       
       
      Show all issues

      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:
    Completed
    Change Summary:
    Pushed to master (892c17b)