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

2018-04-19 22:15:52 +08:00
 holy_sin

大家觉得有必要吗

8744 次点击
所在节点    git
71 条回复
lisonfan
2018-04-19 22:28:47 +08:00
1、没有
2、有必要
clino
2018-04-19 22:29:54 +08:00
有必要
原则上每笔提交都要经过至少另外一名同事的 review
kikyous
2018-04-19 22:35:15 +08:00
我们公司用 arc,还有其他工具可以用吗？
clino
2018-04-19 22:39:34 +08:00
我们用 google 的那个 gerrit
smilingsun
2018-04-19 22:42:33 +08:00
@clino 我们也用 gerrit，你们 review implementation 还是 unit test
seaswalker
2018-04-19 22:55:25 +08:00
有
lgh
2018-04-19 23:16:05 +08:00
@clino
@smilingsun
请教二位：你们在 Gerrit 里面新建项目的时候，是怎么从已有仓库复制的？手工 git clone --bare xxx 吗？我现在就这样搞，但感觉很不方便。
est
2018-04-19 23:16:12 +08:00
有必要。gitlab 使用中。
Perry
2018-04-19 23:19:26 +08:00
用的 Bitbucket code review，需要 2 个以上同事批准，有些项目还要确保 Test Coverage 在一个阈值之上
觉得很有必要
li24361
2018-04-19 23:25:23 +08:00
没有
有必要，对提升自己的代码水平和减少公司项目的坑很有帮助
zpf124
2018-04-19 23:33:34 +08:00
没有，
有必要

另外我们连单元测试也不写几个...
tinycold
2018-04-19 23:40:19 +08:00
我们结对编程。 没开玩笑
jadec0der
2018-04-19 23:42:00 +08:00
有，有必要但是付出的代价太大了
huluhulu
2018-04-19 23:42:30 +08:00
非常有必要，有，使用 gerrit，keyuser 有权限+2，且触发自动编译和自动化 case，都过了才会真正合到 trunk
kingcos
2018-04-19 23:46:15 +08:00
有，2 个看过 Review 过才可以，而且最近正在配置 CI …
xiaojie668329
2018-04-20 00:13:15 +08:00
有必要。准备在例会时提出并推动。
huntzhan
2018-04-20 00:19:13 +08:00
Required, 必须有人 review 才能 check in
Pastsong
2018-04-20 00:20:59 +08:00
有
不强制

是 Proof of Work 的一种方法
PHPer233
2018-04-20 00:21:10 +08:00
管你怎么写，写出来能跑就行。完成任务，交差回家~
Guaidaodl
2018-04-20 00:24:10 +08:00
有啊. 绝对有必要.

而且对新人来说有人 Review 你成长才快.

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

https://www.v2ex.com/t/448284

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

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

© 2021 V2EX