你们都是怎 code review 的？

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 2743 天前的主题，其中的信息可能已经有所发展或是发生改变。

各位 V 友来说说，你是在工作中是怎么 code review 的吧？

第 1 条附言 · 2017-05-21 13:26:14 +08:00

我们有一个用一个 git 的插件，提交的时候，会自动在系统上发起 review。review 的人 accept 了以后，才会把你的代码合到主分支里面去。

Review

Code

工作

怎

12 条回复 • 2017-05-22 20:37:11 +08:00

caixiexin

2017-05-21 12:56:40 +08:00 via Android

上家公司一开始有，后来换领导就没了。
就是各自上台把自己最近做的代码讲一下，特别是核心模块。下面的人有意见会提出来。
review 还是可以学到东西的，要不然天天闷头写代码，不交流一下很多技巧都不知道。

wph95

2017-05-21 14:32:24 +08:00

phabricator

yidinghe

2017-05-21 14:42:03 +08:00 via Android

频繁出问题才考虑代码复审。

hiluluke

2017-05-21 15:57:35 +08:00

提 pull request，没有人 view 是不能合并到 master 的。

kaneg

2017-05-21 16:40:11 +08:00 via iPhone

gerrit

luxinxin

2017-05-21 16:45:24 +08:00

每个代码目录下有一个 owner （或者父目录的 owner ），owner 通过才能提交。

chiu

2017-05-21 16:53:56 +08:00

有新的大功能的话，编码后会组织相关人员，组长等人开 review 会，通过后才集成

SourceMan

2017-05-21 18:29:38 +08:00 via iPhone

pr 工作流

skadi

2017-05-22 08:33:32 +08:00 via Android

pr 呗，然后审查。我有个学长给我讲他公司提交的代码首先要通过脚本的风格审查，比如命名或者其他规范，然后自动跑测试。通过了才给上面人审查，最后才是合并。感觉我司鶸到就只有一步。

qianguozheng

2017-05-22 10:26:00 +08:00

@skadi 我觉得这样挺好

jadetang

2017-05-22 11:34:36 +08:00

@skadi 什么公司？为什么我经历过的公司都没有这种。

zergmk2

2017-05-22 20:37:11 +08:00 via Android

bitbucket