大家有没有尝试过用什么工具提升团队 Code Review 时候的协作效率?

2022-01-11 17:24:07 +08:00
 leontung

因为我之前和现在的公司都实施 Code Review ,我发现我会重复做以下几件事:

  1. 最常见的:在 IDE 里写代码的时候,切换窗口到浏览器打开 Github/Gitlab 的网站,找到自己的 PR ,review 代码提交评论,再回到 IDE 里。
  2. 较为少的:我要 checkout 别人的分支在本地预览代码,这时候我需要 git stash 、git checkout branch 、git pull ,对着 IDE 在浏览器里提交评论。结束之后,checkout 自己开发的分支,恢复 stash 的代码,回忆自己上次写到哪里打开那个文件继续写。

每次做都会耗费我一些时间,这些操作感觉有点琐碎,不知道是否可以在一个地方统一做掉省的来回切了。不知道大家有没有类似的感受?是否有工具可以帮助解决?

5405 次点击
所在节点    程序员
37 条回复
SoloCompany
2022-01-11 19:30:10 +08:00
用 worktree 就可以
aircjm
2022-01-11 19:32:08 +08:00
upsource
Ljcbaby
2022-01-11 22:53:00 +08:00
VSCode 有 Github 插件来着
DuDuDu0o0
2022-01-11 23:48:40 +08:00
@aircjm 查了一下 upsource ,没明白这个软件和 gitlab 上提个 Merge request 有什么区别?
shadowfish0
2022-01-12 00:15:44 +08:00
阿里云效好像有一个命令行方案,可以优化这个流程,名字忘了
Mithril
2022-01-12 00:54:49 +08:00
@DuDuDu0o0 和 JB 家的 IDE 集成的更好一些。
但实际上最关键的还是要减少每次 Review 的代码量,尽可能细的拆分功能。
不然一次搞上去十几个类,几百行代码,折腾啥工具都没用。
代码足够少你用 Gitlab 的 Review 都够了
Goooler
2022-01-12 03:19:53 +08:00
idea 不是自带 GitHub 的代码审查功能,很好用啊
marat1ren
2022-01-12 04:52:21 +08:00
用 gerrit ,直接在网页上 review
aircjm
2022-01-12 07:12:21 +08:00
@DuDuDu0o0 修改建议可以直接打在对应代码上 提高 review 效率啊 特别是很多文件提交的情况下 有上下文 可以直接在 jetbrains 的 IDE 里面进行 review 操作 merge request 光在网页上看 效率极低
corningsun
2022-01-12 07:51:11 +08:00
2 你可以再 clone 一份代码到本地,专门做 code review ;这样不会打断自己写的分支。
LeslieLeung
2022-01-12 08:06:30 +08:00
Phabricator ,有个自带的 arc 工具用来 diff ,然后能生成一个链接,对方用链接就能 comment/accept 你的代码
rhacker1995
2022-01-12 08:24:20 +08:00
idea 上已经集成了 github 的 pr review 了
weiasd
2022-01-12 08:55:25 +08:00
gerrit 是不是已经过时了?
leontung
2022-01-12 10:18:01 +08:00
@aircjm 我昨天找了我朋友请教,他和你持一样的观点,指出 code review 的时候,上下文不一定都在这个 PR 改动的文件中,导致他需要拉到本地用 IDE 看,然后再对照着 IDE 去 Gitlab 里评论。
lululau
2022-01-12 10:30:41 +08:00
团队平均水平不够的话不要做 Code Review ,纯粹浪费时间
EastLord
2022-01-12 10:40:54 +08:00
试试 CodeStream 插件?
leontung
2022-01-12 10:44:21 +08:00
@EastLord 感谢推荐,看着像是很不错,正在看它和 Github 插件的区别。请教一下你团队在用么?
leontung
2022-01-12 10:47:40 +08:00
@corningsun 厉害了,这也是个办法
EastLord
2022-01-12 11:42:28 +08:00
@leontung 我也是刚查到的 团队没在用
connection
2022-01-12 11:47:45 +08:00
可以请教一下你们分支管理以及 CR 时机么

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

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

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

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

© 2021 V2EX