你们公司是 code review 怎么做的的?
1
xmumiffy 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 提醒都解决,基本也没什么问题了。 |