When to do code reviews when doing continuous integration?

Posted by SpecialEd on Programmers See other posts from Programmers or by SpecialEd
Published on 2011-11-08T02:50:13Z Indexed on 2011/11/25 10:04 UTC
Read the original article Hit count: 659

We are trying to switch to a continuous integration environment but are not sure when to do code reviews. From what I've read of continuous integration, we should be attempting to check in code as often as multiple times a day. I assume, this even means for features that are not yet complete.

So the question is, when do we do the code reviews?

We can't do it before we check in the code, because that would slow down the process where we will not be able to do daily checkins, let alone multiple checkins per day.

Also, if the code we are checking in merely compiles but is not feature complete, doing a code review then is pointless, as most code reviews are best done as the feature is finalized. Does this mean we should do code reviews when a feature is completed, but that unreviewed code will get into the repository?

© Programmers or respective owner

Related posts about continuous-integration

Related posts about code-reviews