代码评审是干什么的
团队里的代码评审(Code Review),做的是【0】。
评审最主要的收益
做代码评审,最主要的收益是【0】。
评审该看重什么
一次好的评审,重点该放在【0】。
对事不对人指什么
评审讲究「对事不对人」,意思是【0】。
评审和测试的关系
代码评审和自动化测试的关系是【0】。
这条变更改了多少行
按变更模型,一个 PR 改了 a.py(+10/-3)、b.py(+5/-0),total_changes 交回几? 贯穿本节的变更/PR 模型:files={文件名: (加了几行, 删了几行)}。total_changes 交回改动总行数
PR/MR 是什么
提出 → 有人看提意见 → 批准合并 协作平台上的 PR(合并请求,也叫 MR),本质是【0】。 贯穿本节的变更/PR 模型:files={文件名: (加了几行, 删了几行)}。total_changes 交回改动总行数、files_tou
PR 里评审看什么
提出 → 有人看提意见 → 批准合并 评审者打开一个 PR,主要看的是【0】。 贯穿本节的变更/PR 模型:files={文件名: (加了几行, 删了几行)}。total_changes 交回改动总行数、files_touched 改了几个
PR 什么时候能合
一个 PR 达到可合并,通常要求【0】。 贯穿本节的变更/PR 模型:files={文件名: (加了几行, 删了几行)}。total_changes 交回改动总行数、files_touched 改了几个文件;mergeable(批准数, 改
改动拆小一点好在哪
把一个大改动拆成几个小 PR,好处是【0】。 贯穿本节的变更/PR 模型:files={文件名: (加了几行, 删了几行)}。total_changes 交回改动总行数、files_touched 改了几个文件;mergeable(批准数,