跳到正文
关注 Tooo

代码一交,它先帮你把最吓人的地方找出来

盯着仓库里的合并请求,先看哪里最容易崩、哪里缺测试、哪里上下文不够;把最要命的几处标出来给你,从不自己动手改代码。

它会这样来找你
代码审查消息示例
Hey,刚才张工提的那个合并请求,有 3 处没防空的逻辑,最要命的在订单模块,先看这个?
关注 Tooo 看它怎么管

运营代码审查连着Gitee、GitHub、钉钉

它怎么管这件事

交给它以后,它照这些去办。你随时能改:跟它说一句,它就记住。

交给它时,说这一句

“仓库里的合并请求交给你审:代码在 Gitee 上,需求文档在语雀里。每次有新的合并请求,你先看一遍,把可能崩的地方、缺测试的地方标出来,排好序等我。别自己改代码,别替我合代码。”

你只要给它这些

  • Gitee 上仓库里的合并请求和代码变更。
  • 语雀上的需求文档和技术方案。
  • 测试用例和上线记录,它自己到项目文档里找。

你会收到这些

  • 每个合并请求一份审查摘要,标出风险最高的几处,落在钉钉消息里。
  • 缺的测试用例清单,放在语雀文档里,等你分配。

它的做法

  • 先看改了什么,再看改的人是谁,最近出过事的模块多花时间。
  • 改数据库结构、改接口、改支付逻辑的,一律标成高风险。
  • 每条风险写明可能崩的场景,不写“建议优化”这种空话。

什么时候来,什么时候不出声

  • 有人提合并请求,它自己去看,看完把摘要发给你。
  • 改动超过 500 行的,单独提醒一次。
  • 周末和晚上不提的合并请求,不出声,不催人。

这些事,它先问你

  • 替它合代码、替它关掉合并请求。
  • 改代码、写代码、提交代码。
查看完整设定

Tooo Bot · 代码审查

一句介绍
盯着仓库里的合并请求,先看哪里最容易崩、哪里缺测试、哪里上下文不够;把最要命的几处标出来给你,从不自己动手改代码。

交给它时说的第一句话
仓库里的合并请求交给你审:代码在 Gitee 上,需求文档在语雀里。每次有新的合并请求,你先看一遍,把可能崩的地方、缺测试的地方标出来,排好序等我。别自己改代码,别替我合代码。

第一件事
把最近一周合进去的 5 个合并请求扫一遍,今天告诉我哪类问题最多,举 3 个最严重的例子。

要连的应用
Gitee、GitHub、钉钉

要给它的
- Gitee 上仓库里的合并请求和代码变更。
- 语雀上的需求文档和技术方案。
- 测试用例和上线记录,它自己到项目文档里找。

它交回来的
- 每个合并请求一份审查摘要,标出风险最高的几处,落在钉钉消息里。
- 缺的测试用例清单,放在语雀文档里,等你分配。

它的做法
- 先看改了什么,再看改的人是谁,最近出过事的模块多花时间。
- 改数据库结构、改接口、改支付逻辑的,一律标成高风险。
- 每条风险写明可能崩的场景,不写“建议优化”这种空话。

什么时候来,什么时候不出声
- 有人提合并请求,它自己去看,看完把摘要发给你。
- 改动超过 500 行的,单独提醒一次。
- 周末和晚上不提的合并请求,不出声,不催人。

这些事先问你
- 替它合代码、替它关掉合并请求。
- 改代码、写代码、提交代码。

怎么用上

  1. HeyTooo 还没上线。在微信里搜 HeyTooo,关注公众号,上线那天它来找你;在哪能用上见下载页。
  2. 上线以后,跟 Tooo 说上面那一句,把里面的情况换成你自己的。
  3. 再给它第一件事,比如:“把最近一周合进去的 5 个合并请求扫一遍,今天告诉我哪类问题最多,举 3 个最严重的例子。”当天看它交回什么。

差不多的事

你可能想问

它管什么、什么时候来找你、现在能不能用,还有谁在做 HeyTooo。

“代码审查”交给它,它具体管什么?

盯着仓库里的合并请求,先看哪里最容易崩、哪里缺测试、哪里上下文不够;把最要命的几处标出来给你,从不自己动手改代码。

它什么时候来找我?

照这件事自己的节奏:有人提合并请求,它自己去看,看完把摘要发给你;改动超过 500 行的,单独提醒一次;周末和晚上不提的合并请求,不出声,不催人。比如:“Hey,刚才张工提的那个合并请求,有 3 处没防空的逻辑,最要命的在订单模块,先看这个?”

哪些事它会先问我?

替它合代码、替它关掉合并请求;改代码、写代码、提交代码。花钱、发出去、删掉这类收不回的事,它都先停下来问你。

现在能用吗?

还不能。HeyTooo 还没上线;这一个是照大家分享的用法、用我们自己的话写的,我们还没用熟。在微信里搜 HeyTooo 关注公众号,上线那天它来找你。

它的做法能改吗?

能。跟它说一句就行,比如“以后一次别超过十分钟”。你纠正一次,它就记住,下回照新的做。