请问大家的公司有代码 review 吗

[holy_sin]

大家觉得有必要吗

[EanCuznaivy]

直接 review PR，然后老板亲自 merge ……
很有必要。

[vakara]

有，我们组内强制，Review 很仔细。
公司使用 GitHub。

[longxboy]

有的，强制 review 加各种静态检查编译检查

[tonghuashuai]

有
有必要
强制至少一位同事 review 后才可以 merge

[lsmgeb89]

有必要啊，没有 review 的公司或组就没必要做了。

[ioioioioioioi]

有必要的，不然容易有 bug。

[yuxiaofei93]

公司里有代码审查，并且出事故代码审查人受同样处罚。
代码审查有必要，至少能避免一眼就看出来的低级失误

[unidentifiedme]

有，基本上所有项目强制 code review
很有必要。
所有项目都接 Coverity
有的项目还有 unit test coverage 的要求

[cccRaim]

有，
三个人 review 才行

[shaobin0604]

经历过几家公司，都有 code review，工具
- gerrit
- reviewboard
- gitlab merge request

[coderluan]

公司没有，各个项目自己要求，肯定是有必要的，但是如果一个项目是时间紧张收益低的一锤子买卖，你懂的。

[bluefalconjun]

有必要.
gerrit 提交:
脚本会检查 1.代码格式和 commit 关键字; 2.编译;
然后+1;
另外需要其他队员 review+1;
最后 merge 权限的人才能 merge 进 master.

[jasonchen168]

小公司业务代码都写不完。。。

[ihaveadrame]

不要问题了 选 C。

我们公司对这块要求比较严格，基本是 pylint flake8 eslint 代码规范检测 tox 单元测试 还有 代码覆盖率测试。 每个 PR 都至少要组长 review 一下。

[netlxl]

@Acheron 在最终代码中“偏离代码本质的东西”是没必要的，但成为最终代码的过程中是绝对有必要的。你这是典型的唯结果论。

[kaedea]

1. 本来十几人的团队是互相 rv 的。
2. 后来同一个项目多了几个这种规模的团队。
3. 感到他们的代码风格与自己的格格不入后，慢慢放弃了 rv，因为就算提修改建议人家也懒得改。
4. 现在已经没有定期看别人提交了什么代码的习惯了。

[Deeer]

我遇到的 review 分为两种、一种是成员代码合并 dev 上进行 review、第二种是从 dev 合并到 test 上的 review、一种是确认代码合并时是否存问题、另一种是合并之后进行检查、不知道你说的是哪种 review ？

[kaedea]

@kaedea
补充一下，现在有些人甚至在提交代码前都不 rv 自己提交了啥… 而且还不少这样的人

[xrlin]

看了大家的回答，我要争取去一个注重代码质量的团队，我很想执行 review ci，但是项目就只有我一个人。

[Ranler]

有必要

同 Gerrit，强制 2 个人 Review，强制单元测试 PASS 才能合入