4 ms·

Code reviews must have a higher priority than new development. In a team it should complete within hours. The process also needs to be opinion free. There shoul

Code reviews must have a higher priority than new development. In a team it should complete within hours. The process also needs to be opinion free. There should be a checklist to go through. A code review is not the place to talk about other potential solutions. Also, absolutely no testing during reviews.

agentwiggles 4y ago
> Also, absolutely no testing during reviews. Really? I consider it part of the due diligence of a good review to pull down the branch and at least make sure the tests pass, maybe even try the functionality out if it's something easy to do. Maybe you mean something different in this context though?
- AtNightWeCode 4y ago
  No. Devs should not test other coders code. Much better to hand it over to dedicated testers or the quality assurance team.