发起个讨论，你们公司有 code review 吗？

2016-07-11 10:02:22 +08:00
 zzzzzy

code review 好处很多，可以规范代码、传递知识和保证代码质量等，但是因为项目进度和其他因素，不一定保证 review 被有效执行。请教各路大神都是怎么做的？有没有啥套路、工具、方法之类的。

16334 次点击
所在节点    程序员
108 条回复
YORYOR
2016-07-12 13:04:15 +08:00
上线前 必须 review
eliteYang
2016-07-12 13:34:08 +08:00
不算是特别规定的 code review,是负责人会定期查看。
查到问题，基本是说出来就要立刻改掉。
有一些静态检查工具，以及看修改的内容对比。

code review 本来就是一个用人来检查的活儿，如果有工具可以做，就基本用不到人了，所以暂时没啥特别的套路和工具。
wohenyingyu01
2016-07-12 13:37:08 +08:00
只有我一个人用 OC ，没人看得懂
nikola
2016-07-12 13:56:53 +08:00
自己 review 自己代码算吗？一天产生那么多代码，谁有空来 review ？
iphantom
2016-07-12 14:51:00 +08:00
@hantsy 哈哈 外企
summersky92
2016-07-12 15:53:39 +08:00
每行代码都要 review 一遍才算这个 task 完成。。。也不是很大的公司
但确实仔细 review 过的代码 merge,deploy 的时候问题比较少
确实很费时 但是不管是别人 review 你的代码还是你 review 别人的代码都是一个互相学习的过程
hmckkk
2016-07-12 17:41:49 +08:00
提交的 feature 分支全部走 gitlab 的 MR ， review 完才合并回主开发分支。
kinghui
2016-07-12 17:57:17 +08:00
Github 长期分支设置保护, 只能通过 Fork + PR 提交进行 Review, 要求必须写单元测试. Review 完交给持续集成跑单元测试和代码检测并检查测试覆盖率. 之后机器人( https://github.com/barosl/homu)会根据持续集成返回的结果来合并 PR.

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://yangjunhui.monster/t/291623

V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX