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

@ourcubk Github 做 Code Review 不是更直接。。。

peer to peer review.

@clorts 偷懒的好理由。。。

之前小公司，基本上自己做一個 project ，別人也 review 不了。現在做的開源項目， 100+ contributors 。沒有 code review 和 CI 早就掛了吧。

只发生在找 BUG 的时候

@hantsy 单元测试么？ 这个要写的，需要覆盖到。

我们公司每周都要 review 代码，我觉的这样我们大代码拿出来被大家看看才会知道自己哪里需要改进，这样我们才会有进步的啊！

每次 merge 必须 review

搭车问有哪些好的 review 的工具 :)

没有。

上周刚从 SVN 切换 到 Git 。

后端开发至少 7 人，前端开发至少 9 人。

开发完成后自己提交一个 MR ， leader 会 review ，通过了再 merge ，否则打回改到符合要求。项目稳定之后这么执行其实很好的，整套流程配合起来很顺畅

这么说吧。 外企基本上都有 review 的， 而且是算工作量的。 Reviewer 需要承担责任。 所以一般会比较认真 ， 貌似看起来比较费时间，但是带来的产品质量提升很有效。 也少了给项目埋地雷的机会。。

国内很多公司急功近利，不重视这个， 总归是要走弯路的， 修不完的 bug 。。

都说会有，但是真正做起来的很少，哎

我们老大会看每次提交，一些修改会做 review ，也会对这些修改进行测试。我觉得 review 是必须的，但是 reviewer 需要是有经验有实力的 coder ，要对整个系统很熟悉。经常会给我们指出错误和提出优化意见，很棒。

@iphantom 看来国内还是有那么 10%的公司重视软件开发质量的。。。

我们公司都是每个组组员之间互相 Review 。

有，我负责 review ，我最喜欢喷人了

不定期 review

review 就跟改作业一样，老师不帮你改作业，你难道不慌嘛

有啊，在外企。经常不到１０行代码 review 好几个来回耗时几天，各种挑毛病，实在没大毛病还得挑变量名啊， trailing spaces 啊之类的，痛苦得扭来扭去。