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

用的 git 每次都需要 review 然后再合并
我感觉 leader 只看看大概逻辑和写代码风格
特别是有无性能特低的代码，有没有良好的缩进和注释习惯

外包行业，做项目的时间都不够，唉，还 review 。

@proudzhu 因为是 test fixture.....

有，个人觉得很重要
我们公司时间也很紧，天天晚上加班
然而 CR 流程一直保留

@Tonni github enterprise 价格多少？

@Ixizi review 代码要看多久？
@broadliyn review 是闲的蛋疼公司才干的么？功能都赶不及， review 个蛋啊：）
@yangxiongwei 用这个多久了？是不是很烦躁呢？
@repus911 review 这么麻烦啊？是一行行代码看过去么？都能看懂么？

有，必须有人 review 才能 merge 。

@clorts gitlab 我们 gitlab 用开源版 还没有特殊需求

有的大功能比较麻烦，数据库看一遍， redis 看一遍，核心逻辑看一遍，兼容性过一遍，确认一遍。中间还有打回去修改，那边一个 sprint 10 天开发过来的 mr 怎么也得对上个 1 、 2 个小时。

当然这种情况比较少见，一般 bugfix 10 分钟，小功能半个小时搞定，还有熟练工发过来的来基本也是 10 分钟内。

别说我还真遇见过单元测试和 QA 没覆盖到的低级语法错误，嘿嘿嘿。

@clorts 习惯就好。代码质量就靠这个加 测试来保证了。

而且这样是非常有必要的，前期花点功夫，后期维护起来会简单很多。

@sc3263 svn 可以和 review board 配合起来用，很方便

@chousb one in One?

没有啊

新功能没有。旧的代码，涉及底层公共的一般我自己写，让小弟看看久合了。剩下的就让测试妹子把关了。

代码->gerrit->自动编译 /UT/复杂度校验->review->review 通过触发自动编译生成 img->自动生成包->自动安装包->重启机器->启动完成后触发各种 case 跑自动化测试->代码 gerrit merge into git brunch->git brunch 自动 merge into brunch->终于他妈妈的完成了...........................................................................................................
review 都是小事,重要的耗时间的都是 review 无关,却严格控制代码之类的其他工作...我们公司这样进去的代码,感觉还挺放心的...

git brunch 自动 merge into master...

再来一个话题，有写 test 吗？

代码烂成屎，入错坑了。。

@mahone3297 哈哈哈哈 没有

@kqz901002 是么？我试试。谢啦~

其实讲道理，这东西，得项目组 leader 规定，强制执行才有效果。个人提出来，只会被认为工作量不饱和，闲着没事干。前几天在新项目组里提出来要搞一整套流程。然后被所有人反对了，理由是浪费时间。。。

@iphantom 你们不写测试吗》

基本上用 Github Flow 。。。 Fork ， Branch, PR, Code Review

结合 Circle CI 等，每个 Branch 都是自动运行测试，从建 PR 开始几乎团队所有人都会参与 Review ，特别与其他人有交叉的部分，@相关人，反复讨论修改。

Upstream Master 在一些项目是稳定分支，执行自动部署。没 Code Review 谁敢直接部署？