这是一个创建于 3165 天前的主题，其中的信息可能已经有所发展或是发生改变。
各位 V 友 来说说，你是在工作中是怎么 code review 的吧？
第 1 条附言 · 2017 年 5 月 21 日
我们有一个用 一个 git 的插件，提交的时候，会自动在系统上发起 review。review 的人 accept 了以后，才会把你的代码合到主分支里面去。
 |
1
caixiexin 2017 年 5 月 21 日 via Android
上家公司一开始有，后来换领导就没了。
就是各自上台把自己最近做的代码讲一下，特别是核心模块。下面的人有意见会提出来。 review 还是可以学到东西的，要不然天天闷头写代码，不交流一下很多技巧都不知道。 |
 |
2
wph95 2017 年 5 月 21 日
phabricator
|
 |
3
yidinghe 2017 年 5 月 21 日 via Android
频繁出问题才考虑代码复审。
|
 |
4
hiluluke 2017 年 5 月 21 日
提 pull request，没有人 view 是不能合并到 master 的。
|
 |
5
kaneg 2017 年 5 月 21 日 via iPhone
gerrit
|
 |
6
luxinxin 2017 年 5 月 21 日
每个代码目录下有一个 owner （或者父目录的 owner ），owner 通过才能提交。
|
 |
7
chiu 2017 年 5 月 21 日
有新的大功能的话，编码后会组织相关人员，组长等人开 review 会，通过后才集成
|
 |
8
SourceMan 2017 年 5 月 21 日 via iPhone
pr 工作流
|
 |
9
skadi 2017 年 5 月 22 日 via Android
pr 呗，然后审查。我有个学长给我讲他公司提交的代码首先要通过脚本的风格审查，比如命名或者其他规范，然后自动跑测试。通过了才给上面人审查，最后才是合并。感觉我司鶸到就只有一步。
|
 |
10
qianguozheng 2017 年 5 月 22 日
@skadi 我觉得这样挺好
|
 |
12
zergmk2 2017 年 5 月 22 日 via Android
bitbucket
|
Select Language