这是一个创建于 1607 天前的主题，其中的信息可能已经有所发展或是发生改变。
你们公司是 code review 怎么做的的？
 | 1 106npo 2021-05-16 18:18:46 +08:00 via Android merge 前在 merge request 中 review |
 | 2 duhb 2021-05-16 18:26:53 +08:00 我们公司不 code review |
 | 5 tychio 2021-05-16 20:27:18 +08:00 用这个 https://www.sourcetreeapp.com/ 很方便，可以逐条 commit 的 review 。是谁提交的谁就给其他人解释一下思路，然后讨论。每天下午把当天的代码都过一下就行了，有问题记下，回头改了的 commit 自然会出现在下次 review 的过程中 |
 | 6 liuhuansir 2021-05-16 20:59:44 +08:00 @honglei92 code review 还能自动化？多数都是业务代码，怎么自动化？ |
 | 8 youngce 2021-05-16 21:55:24 +08:00 闲的时候，提交代码 merge 时，会有人 code review 。后来面向 KPI 编程，需求多的时候，就没有 code review 啦，都太忙了，帮别人 review 代码 |
 | 10 yazoox 2021-05-16 22:45:47 +08:00 没看懂！ 从每个人的开发分支合并到 master/main 主分支时，可以创建 pull request 啊？那个是可以设置强制有人（多人） code review 和 approve，才可以合并的。 现在有很多工具可以 hook 到 git 上面去，做一些自动化代码审查 |
 | 11 witcherhope 2021-05-17 09:04:54 +08:00 via iPhone 加代码扫描和覆盖率扫描钩子 |
 | 12 honglei92 OP @witcherhope 可以具体点吗？覆盖率扫描是啥？ |
 | 13 nicocho 2021-05-17 10:15:24 +08:00 人工.. 提交代码后，会有人简单看下流程逻辑，没问题就帮忙合进去，后来忙起来，都不管不顾了。 |
 | 14 kingiis 2021-05-17 10:16:50 +08:00 太理想了 工期正常时 格式很工整,忙的时候直接放飞自我了 |
 | 15 neptuno 2021-05-17 10:22:24 +08:00 提个 pr，找几个业务相关的同事+TL 慢慢看呗，kpi 就是最好的自动化，c 拿多了，代码自然就好了 |
 | 16 xz410236056 2021-05-17 10:33:23 +08:00 大公司直接花钱买 sonar 了，各种报告非常详细，能具体到某一行代码，没钱别那么多事。 |
 | 17 unco020511 2021-05-17 10:37:35 +08:00 一般就是 pr 时 code review 吧 |
 | 18 ychost 2021-05-17 16:52:06 +08:00 PR 逐条看代码 |
 | 19 en20 2021-05-17 17:43:10 +08:00 我给你纠正一下, 如何优雅"地" |
 | 21 magicls 2021-05-24 10:23:50 +08:00 部署 Gerrit + Jenkins，每次提交自动跑 Lint 。 你能把 Lint 跑出来的 Java 跟 Android 提醒都解决，基本也没什么问题了。 |
Select Language