强制 code review reviewboard+svn 的方案

分享一下我老师大神的人工智能教程！零基础，通俗易懂！http://blog.csdn.net/jiangjunshow

也欢迎大家转载本篇文章。分享知识，造福人民，实现我们中华民族伟大复兴！

               

赖勇浩（http://laiyonghao.com）

2012 年 2 月 22 日更新：保持文档与 0.1.2-r14 版本同步（变更部分使用蓝色#3333FF）。

2012 年 3 月 8 日更新：保持文档与 0.1.2-r16 版本同步（变更部分使用红色#CC0000）。

2012 年 4 月 28 日更新：保持文档与 0.1.3-r16 版本同步（变更部分使用深紫色#993399）。

2013 年 10 月 17 日更新：保持文档与 0.2.1-r20 版本同步（变更部分使用洋红色#FF6666）。

我们团队在开发《天下盛境》项目的时候，制定和执行了比较好的 code review 策略，总结下来有几个优点：一是代码风格可控，代码质量有一定提升；二是新员工入职后能够得到更多人的指导，成长非常快；三是小 bug 频出的情况比我做《天》之前的项目少了至少一个数量组。当时我们的 code review 策略是这样的：

1. 使用 reviewboard 作为工具，通过 SVN hooks 强制每一次签入都是经过 review 的；
2. 至少要有 2 个团队成员 ship it，才能够签入。
3. ship it 的成员中，至少有一个是资深的团队成员。

code review 是如此的有效，以至于我经常向朋友推荐，有一些朋友使用之后，觉得把 reviewboard 跟 SVN 结果起来还是蛮有挑战的，主要是编写 SVN hooks 还是需要学习不少东西，