知识卡片
评审先看流水线
内容
合并请求评审前应先展示代码扫描、单元测试等自动化结果;如果基础测试没通过,人工评审就容易浪费。发散:机器先筛掉低级问题,人再判断高阶意图。
参考来源
- 位置:《软件交付通识》第22章《代码评审》(源文件:_epub-src/text/part0028.html)
- 结论依据:原文说评审工具应知道代码扫描、单元测试等自动化情况,若未通过就没必要人工评审。
- 原始内容:说到与代码评审相关的工具间集成,首先要提两个基础能力:一是知道这次代码评审所评审的改动的目的,其体现在代码评审说明、合并请求说明、代码改动提交说明、特性分支名称上,并且最好是链接到相关工作项;二是知道代码扫描、单元测试等自动化测试的情况,如果它们都没通过,那么就没必要进行人工评审了。