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