• 
      

    Google Code post-receive hook

    Review Request #5744 — Created April 26, 2014 and submitted

    Information

    Review Board
    master
    d232545...

    Reviewers

    This is a Google Code post-receive hook that will automatically close review requests as "submitted" after a push. To determine which review requests should be closed, it scans through each commit's commit message for the following strings (case-insensitive): "Reviewed at <reviewboard_url>/r/<id>" or "Review request #<id>".

    This is what the JSON payload looks like:
    http://code.google.com/p/support/wiki/PostCommitWebHooks

    All tests performed using an SVN and Mercurial repositories.

    1) Push with the review request id in the commit message. Review request closed. [PASSED]
    2) Push without a review request id in the commit message. (Error: Review request does not exist) [PASSED].
    3) Push with non existent review request id. (Error: Review request does not exist) [PASSED]
    4) Push with review id in the message that is already submitted. (logged a warning: request id already submitted.) [PASSED]
    5) Push with review id in the message that has been marked as discarded. (review request marked as submitted.) [PASSED]

    Description From Last Updated

    Can you call it "Google Code" instead of "googlecode"?

    daviddavid

    There's an extra period before the : in this message. Please also add exc_info=1 to the logging call.

    daviddavid

    Please add exc_info=1 to the logging call.

    daviddavid

    Can you make this text a comment rather than part of the docstring? It's less about describing the behavior of …

    daviddavid

    Can you capitalize "SVN" and "Mercurial" in here?

    daviddavid
    B.
    david
    1. 
        
    2. reviewboard/hostingsvcs/googlecode.py (Diff revision 2)
       
       
       
       
      Show all issues

      Can you call it "Google Code" instead of "googlecode"?

    3. 
        
    B.
    B.
    david
    1. 
        
    2. reviewboard/hostingsvcs/googlecode.py (Diff revision 3)
       
       
      Show all issues

      There's an extra period before the : in this message.

      Please also add exc_info=1 to the logging call.

    3. reviewboard/hostingsvcs/googlecode.py (Diff revision 3)
       
       
      Show all issues

      Please add exc_info=1 to the logging call.

    4. reviewboard/hostingsvcs/googlecode.py (Diff revision 3)
       
       
       
       
       
       
      Show all issues

      Can you make this text a comment rather than part of the docstring? It's less about describing the behavior of the method and more about the implementation of it.

    5. reviewboard/hostingsvcs/googlecode.py (Diff revision 3)
       
       
       
       
      Show all issues

      Can you capitalize "SVN" and "Mercurial" in here?

    6. 
        
    B.
    B.
    david
    1. Ship It!

    2. 
        
    B.
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.0.x (ccc1565)