How can I coordinate code review tool and RCS (specifically git)

Posted by Chris Nelson on Stack Overflow See other posts from Stack Overflow or by Chris Nelson
Published on 2010-04-13T19:25:48Z Indexed on 2010/04/14 14:03 UTC
Read the original article Hit count: 440

Filed under:
|
|
|

We're committed to git for code management. We're trying to find a tool that will help us systematize code reviews. We're considering Gerrit and Code Collaborator but would welcome other suggestions.

We're having a problem answering the question, "How do we know every commit was reviewed?" (Or "What commits have yet to be reviewed?")

One answer would be to submit every commit or every push for review and track incomplete reviews in the review tool. I'm not entirely happy with relying on a another tool -- especially if it's not open source -- to tell us this.

What seems to be a better answer is to rely on sign offs in git (e.g., "Signed-off-by: Chris Nelson") and use a hook in the review tool to sign off commits on behalf of the reviewer. And advantage of this is if we use some other review mechanism for some commits, we have just one place to look for results. One problem with this is that we can't require review before push because the review tool is unlikely to have access to the developer's private repository clone to add the sign-off.

Any ideas on integrating code review with code management to achieve ease of use and high visibility of unreviewed changes?

© Stack Overflow or respective owner

Related posts about rcs

Related posts about code