大家有没有尝试过用什么工具提升团队 Code Review 时候的协作效率？
leontung · ZhenhangTung · 2022-01-11 17:24:07 +08:00 · 5715 次点击这是一个创建于 1031 天前的主题，其中的信息可能已经有所发展或是发生改变。
因为我之前和现在的公司都实施 Code Review ，我发现我会重复做以下几件事：
- 最常见的：在 IDE 里写代码的时候，切换窗口到浏览器打开 Github/Gitlab 的网站，找到自己的 PR ，review 代码提交评论，再回到 IDE 里。
- 较为少的：我要 checkout 别人的分支在本地预览代码，这时候我需要 git stash 、git checkout branch 、git pull ，对着 IDE 在浏览器里提交评论。结束之后，checkout 自己开发的分支，恢复 stash 的代码，回忆自己上次写到哪里打开那个文件继续写。
每次做都会耗费我一些时间，这些操作感觉有点琐碎，不知道是否可以在一个地方统一做掉省的来回切了。不知道大家有没有类似的感受？是否有工具可以帮助解决？
 |
1
SoloCompany 2022-01-11 19:30:10 +08:00
用 worktree 就可以
|
 |
2
aircjm 2022-01-11 19:32:08 +08:00 via Android
upsource
|
 |
3
Ljcbaby 2022-01-11 22:53:00 +08:00
VSCode 有 Github 插件来着
|
 |
4
DuDuDu0o0 2022-01-11 23:48:40 +08:00
@aircjm 查了一下 upsource ，没明白这个软件和 gitlab 上提个 Merge request 有什么区别？
|
 |
5
shadowfish0 2022-01-12 00:15:44 +08:00
阿里云效好像有一个命令行方案，可以优化这个流程，名字忘了
|
 |
6
Mithril 2022-01-12 00:54:49 +08:00
@DuDuDu0o0 和 JB 家的 IDE 集成的更好一些。
但实际上最关键的还是要减少每次 Review 的代码量，尽可能细的拆分功能。 不然一次搞上去十几个类，几百行代码，折腾啥工具都没用。 代码足够少你用 Gitlab 的 Review 都够了 |
 |
7
Goooler 2022-01-12 03:19:53 +08:00 via Android
idea 不是自带 GitHub 的代码审查功能，很好用啊
|
 |
8
marat1ren 2022-01-12 04:52:21 +08:00 via iPhone
用 gerrit ，直接在网页上 review
|
|
9
aircjm 2022-01-12 07:12:21 +08:00 via Android
@DuDuDu0o0 修改建议可以直接打在对应代码上 提高 review 效率啊 特别是很多文件提交的情况下 有上下文 可以直接在 jetbrains 的 IDE 里面进行 review 操作 merge request 光在网页上看 效率极低
|
 |
10
corningsun 2022-01-12 07:51:11 +08:00 via iPhone
2 你可以再 clone 一份代码到本地，专门做 code review ；这样不会打断自己写的分支。
|
 |
11
LeslieLeung 2022-01-12 08:06:30 +08:00 via iPhone
Phabricator ，有个自带的 arc 工具用来 diff ，然后能生成一个链接，对方用链接就能 comment/accept 你的代码
|
 |
12
rhacker1995 2022-01-12 08:24:20 +08:00
idea 上已经集成了 github 的 pr review 了
|
 |
13
weiasd 2022-01-12 08:55:25 +08:00
gerrit 是不是已经过时了？
|
 |
14
leontung OP @aircjm 我昨天找了我朋友请教，他和你持一样的观点，指出 code review 的时候，上下文不一定都在这个 PR 改动的文件中，导致他需要拉到本地用 IDE 看，然后再对照着 IDE 去 Gitlab 里评论。
|
 |
15
lululau 2022-01-12 10:30:41 +08:00
团队平均水平不够的话不要做 Code Review ，纯粹浪费时间
|
 |
16
EastLord 2022-01-12 10:40:54 +08:00  1
试试 CodeStream 插件？
|
|
18
leontung OP @corningsun 厉害了，这也是个办法
|
 |
20
connection 2022-01-12 11:47:45 +08:00
可以请教一下你们分支管理以及 CR 时机么
|
 |
22
Cihua 2022-01-12 13:46:48 +08:00
不确定是指那种，我们之前 java 用 SonarQube ，在 jenkins 打包镜像过程中去检测，发现不符合定的标准则会打包失败，同时 idea 有插件可以连接 sonar 服务器直接检测。
|
 |
23
lizuoqiang 2022-01-12 14:27:04 +08:00
@aircjm upsource 不支持 gitee 吗
|
|
24
leontung OP @connection 分支管理部分，我们是在一个仓库里开发，不走 fork 路线。主要有 3 大分支：production 、main 和 develop 。大家从 develop 切分支，开发完合并到 develop ，develop 对应 test 环境；准备上线前从 develop 合并 main ，main 对应 staging 环境，业务上这个环境预览测试； main 合并 production 分支，上生产环境。
|
|
25
leontung OP @EastLord thanks ，我看下是否好用，我认为在 IDE 里完成 code review 是挺爽的，但可能只是我的个人观点。最好能够找到让我老板买单的点。
|
|
26
leontung OP @Goooler thanks ，我和楼下提到的 CodeStream 对比一下。想请教一下你是觉得是这个插件好用，还是在 IDE 里 code review 这个过程好用呢？
|
|
28
leontung OP @lizuoqiang 不支持
> Review GitHub pull requests and GitLab merge requests in Upsource. |
|
29
aircjm 2022-01-12 18:45:34 +08:00
@lizuoqiang (⊙﹏⊙) 我们之前用的时候用的是自建的 gitlab 没了解过 gitee 我理解可以用同步的方式同步 git 库 这个应该不是大问题
|
|
30
aircjm 2022-01-12 18:57:41 +08:00
@leontung 换了团队了 需求太多就没时间做 review 😓
说下之前是怎么玩的 他是 10 人免费的 可以只建两个账号 ，一个提交人，一个 review 人 要 review 的时候合并提交记录，尽量在提交记录少的情况下进行 review 。 每个人都可以用可以用这两个账号 但是也出现过账号混乱使用的问题 也可以找 pj 但是不建议这样做 公司用不要用破解东西 有风险 |
 |
31
dayeye2006199 2022-01-13 06:35:46 +08:00
原来只有我一个人永远看的是网页。。
|
|
32
dayeye2006199 2022-01-13 06:41:32 +08:00
我觉得其实工具倒还是次要的。以下几点我觉得对 review 的效率更重要：
1. 每个 PR 只做一件事情 -- 加一个 feature ，修复一个 bug 等等；这样可以保证一次需要 review 的代码量不会太大 2. 一些格式问题，常见的语法问题等等不要人来 review ，用 linter 和分析器来解决，review 之前保证先把这些低级错误给修复了 3. 特别复杂的 PR 需要附上设计文档，作者需要阐述大概是怎么实现这个功能的 4. 测试还是需要写，否则全靠人来把握代码正确性，reviewer 的心理负担会很重 |
 |
33
vilic 2022-01-13 07:33:44 +08:00
我们通过远程工作空间解决这个问题，然后整合到了工作流中，不过也有代价。
方案比较粗糙，因为是内部工具能用就行，可以参考一下。 https://github.com/makeflow/remote-workspace |
|
34
leontung OP @dayeye2006199 哈哈，我也一直看网页，直到想到为什么不在 IDE 里 review ，是不是更高效？所以跑来请教大家意见
|
 |
36
Akiya 2022-01-15 13:41:07 +08:00 via iPad
vscode 的 GitHub pull requests 插件基本上可以满足需求了
|
 |
37
kejin114978 2022-04-13 17:17:46 +08:00 via Android
我们在用云效，开箱即用，还不限制团队人数，背靠阿里大厂😁
|
Select Language