Skip to content

Comment on Things Everyone Should Do: Code Reviewparent

Comments

At work we have just rolled out Gerrit as the review tool. The setup is simply that you push your commits to Gerrit, then reviewers can comment on and approve (or not) the change in the web tool. Reviewers can also fetch your patch and run it on their own system.

After a change has been approved, it is pushed into the CI system by Gerrit, and if/when it passes that it is pushed into the public repos.

Before that all reviews were done by passing around diff files.

There was a long discussion [1], and in the end Gerrit was chosen. I have used Review Board previously, but actually like Gerrit quite a lot now that I'm forced to work with it ;)

[1] http://lists.qt-labs.org/public/opengov/2011-February/000260...

AboutSource Built by g1lg1l

Hackerly is an independent reader for Hacker News, built on the public HN API. Not affiliated with Y Combinator.