• 
      

    Repository check for RBTools command "post"

    Review Request #6798 — Created Jan. 17, 2015 and discarded

    Information

    RBTools
    master

    Reviewers

    When submitting to repository with RBTools, if requested repository could not be found by the Review Board server, raise an Error.
    
     
    Description From Last Updated

    Please revert this changed line.

    daviddavid

    You don't need parentheses around the conditional statement.

    daviddavid

    This should be in sentence casing, and should ideally be fleshed out a bit more. If you were to see …

    chipx86chipx86

    Col: 1 W293 blank line contains whitespace

    reviewbotreviewbot
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          rbtools/commands/post.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          rbtools/commands/post.py
      
      
    2. 
        
    david
    1. 
        
    2. rbtools/commands/post.py (Diff revision 1)
       
       
      Show all issues

      Please revert this changed line.

    3. rbtools/commands/post.py (Diff revision 1)
       
       
      Show all issues

      You don't need parentheses around the conditional statement.

    4. 
        
    chipx86
    1. Make sure you read my recommendations in #students about drafting summaries, descriptions, and testing, and look at some of the examples.

    2. rbtools/commands/post.py (Diff revision 1)
       
       
      Show all issues

      This should be in sentence casing, and should ideally be fleshed out a bit more. If you were to see this from a program you just started using, you'd probably want it to be more explicit about what went wrong.

      It's also helpful to have errors appear more naturally-written, like you were telling it to someone.

    3. 
        
    TI
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          rbtools/commands/post.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          rbtools/commands/post.py
      
      
    2. rbtools/commands/post.py (Diff revision 2)
       
       
      Show all issues
      Col: 1
       W293 blank line contains whitespace
      
    3. 
        
    mike_conley
    1. Hey tienv, any update to this one?

    2. 
        
    david
    1. This should have a bug number assigned. Please also address Review Bot's comment.

    2. 
        
    TI
    Review request changed
    Status:
    Discarded