代码评审容易遗漏哪些问题:从漏检清单到合入前检查
代码评审容易遗漏的往往不是语法错误,而是需求覆盖、异常边界、并发事务、数据迁移、安全权限、接口兼容、测试有效性与评审闭环等需要跨文件判断的问题。本...
覆盖 ONES ISSUE 已发布内容,按主题持续更新。
代码评审容易遗漏的往往不是语法错误,而是需求覆盖、异常边界、并发事务、数据迁移、安全权限、接口兼容、测试有效性与评审闭环等需要跨文件判断的问题。本...
代码评审要落地,需要把一次提交变成有输入、有检查项、有结论、有记录的合入流程。本文按提交到合入的顺序,给出步骤表、可裁剪的检查清单、角色交付物对照...
Code Review是代码合入主干或发布分支之前,由作者以外的开发者检查一次具体变更并给出合入结论的协作活动。本文从定义、边界、环节和相邻概念几个方面展开,...
按提交前自检、评审中核对、合入前确认三个时点组织的Code Review检查清单,每项写明通过标准、证据与负责人,附阻断项清单与合入完成判定条件。
评审人靠临时指定、合入靠口头确认,是Code Review规则失效的常见原因。本文给出评审人分配规则表、合入检查清单、角色分工与周期性检查表,并说明如何把规则...
待审队列指已经提交、正在等待评审或合入处理的变更集合。本文说明它的定义、边界与形成机制,并区分它与个人待办、积压、评审等待指标的不同视角。
合并请求是代码托管平台中的一类正式申请:把某个分支的改动并入目标分支,并在合入前完成检查与评审。本文说明合并请求的定义、组成、从创建到合入的流程,...
代码评审分派要回答谁来看、看什么、多久看完、看完怎么合入。本文给出评审人选择规则表、分派四步操作、角色职责对照与合入条件清单,并说明如何检查分派机...
代码评审分派要明确责任和交付要求,需要把“谁评、评什么、评到什么程度、何时给结论”写进分派规则和评审单。本文给出角色职责表、评审单字段模板、从提交到...
代码评审反馈处理的关键是把每条评论当成有状态的工作项:先分类,再指派负责人和回复口径,以“评论已回复且结论已落地”作为关闭条件。本文给出四步流程、分...