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

[zzzzzy]

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

[zhenjiachen]

技术经理：什么是 code review ？

[bigtan]

完全不需要，自己写，自己用。

[chousb]

One on One Code Review.

[lution]

不同的组不一样吧，反正我们组是必须 review 的。公司层面应该到了一定规模的公司很难规定这个东西。

[Yc1992]

方便某个人离职后其他人迅速交接

[tabris17]

review by myself
功能都来不及赶， review 个毛线

[repus911]

自己搭了个 gitlab 上线开 mr 有权限合的人去 review

[zzzzzy]

@chousb 结对编程前期成本很高吧，还要看团队人员构成，我们这一个老员工， 3 个新来的小鲜肉，怎么破？

[zzzzzy]

@tabris17 业务需求频繁、赶项目，可能确实要 review self

[mrwangrj]

review 邮件 是展示工作量的方式。。。

[zzzzzy]

@zhenjiachen 哈哈

[knightdf]

自写自测自 review 自上线

[zzzzzy]

@lution 基础的、功能性的组件确实要做 review ，请问你们怎么做的？

[zzzzzy]

@Yc1992 你说的好像也有道理，哈哈

[murmur]

每个新人 的第一个模块要 review 的
或者新系统的 每个人的 第一个模块
后面看造化了

[lution]

@zzzzzy 基本就是 7 楼说的那样

[hxtheone]

我们公司是所有的代码都要 review

[zhouxuchen]

刚进公司的时候负责人大概 review 了一个礼拜我的代码（当然到底有没有 review 我就不知道了）。后来一切就看一个人的命运和历史的进程了（

[zvving]

大家对 code reivew 好不重视啊。

我们是用 gitlab - merge request 做的， develop 分支不能直接提交代码，全走 MR ，自己不能 merge 自己发的 MR 。
好处就不说了。

[Phariel]

Atlassian 的 Stash ，还有 Gerrit 都很好用