这是一个创建于 3084 天前的主题，其中的信息可能已经有所发展或是发生改变。
code review 好处很多，可以规范代码、传递知识和保证代码质量等，但是因为项目进度和其他因素，不一定保证 review 被有效执行。请教各路大神都是怎么做的？有没有啥套路、工具、方法之类的。
 |
101
YORYOR 2016-07-12 13:04:15 +08:00
上线前 必须 review
|
 |
102
eliteYang 2016-07-12 13:34:08 +08:00
不算是特别规定的 code review,是负责人会定期查看。
查到问题，基本是说出来就要立刻改掉。 有一些静态检查工具，以及看修改的内容对比。 code review 本来就是一个用人来检查的活儿，如果有工具可以做，就基本用不到人了，所以暂时没啥特别的套路和工具。 |
 |
103
wohenyingyu01 2016-07-12 13:37:08 +08:00 via iPhone
只有我一个人用 OC ，没人看得懂
|
 |
104
nikola 2016-07-12 13:56:53 +08:00
自己 review 自己代码算吗？一天产生那么多代码，谁有空来 review ？
|
 |
106
summersky92 2016-07-12 15:53:39 +08:00
每行代码都要 review 一遍才算这个 task 完成。。。也不是很大的公司
但确实仔细 review 过的代码 merge,deploy 的时候问题比较少 确实很费时 但是不管是别人 review 你的代码还是你 review 别人的代码都是一个互相学习的过程 |
 |
107
hmckkk 2016-07-12 17:41:49 +08:00
提交的 feature 分支全部走 gitlab 的 MR ， review 完才合并回主开发分支。
|
 |
108
kinghui 2016-07-12 17:57:17 +08:00
Github 长期分支设置保护, 只能通过 Fork + PR 提交进行 Review, 要求必须写单元测试. Review 完交给持续集成跑单元测试和代码检测并检查测试覆盖率. 之后机器人( https://github.com/barosl/homu)会根据持续集成返回的结果来合并 PR.
|
Select Language