审查特性分支
审阅功能分支时,以约定基线、项目规则和预期行为为依据。将规范问题与需求缺失或实现错误分别记录,并把每项发现对应到相关修改行。这样作者能够区分代码约定与功能后果,按问题本身安排修复。
使用 Matt Pocock 的 Code Review,对照指定 Git 基准审查分支。分别检查仓库规范与原始需求,并列查看带有代码位置的问题,避免将符合规范误当作需求已经完整实现。
指定固定比较点和修改需要满足的要求,将审阅请求区分为规范检查与行为缺口。发现应关联相关差异及证据,而不是只给出笼统的代码质量印象,方便随后决定具体修复。
请求可预览和复制,不会从此处自动发送。
参考来源: Code Review https://github.com/mattpocock/skills/blob/main/skills/engineering/code-review/SKILL.md 先阅读原始说明并核对当前环境与依赖。需要安装或连接新工具时,先提出方案。 我的任务: 使用 Matt Pocock 的 Code Review 对照 main 审查分支,需求文件是 docs/specs/import.md,分别报告规范与需求发现。 期望交付: 分别报告规范与需求问题,并关联相关修改行。
Code Review 是 Matt Pocock 工程集合中的双轴审查 Skill。它先解析比较参考,确认基于合并基点的差异非空,再查找原始需求与仓库编码规范。两个独立子代理分别检查规范和需求覆盖,最终报告保留两方面发现。规范轴还使用有明确标识的代码异味基线,但项目文档中的约定具有优先权。没有需求说明时,会明确跳过需求轴,而不是自行编造。使用需要可访问的 Git 历史、预期比较点及集合的需求跟踪配置;它提供审查流程,不会自动批准修改,也不保证完成问题修复。
分支规范、任务单要求和发布范围,需要各自清楚的比较基线。Code Review 将发现对应到固定版本、相关差异和支持证据,让审阅者能够区分风格问题、缺失行为与范围遗漏。
审阅功能分支时,以约定基线、项目规则和预期行为为依据。将规范问题与需求缺失或实现错误分别记录,并把每项发现对应到相关修改行。这样作者能够区分代码约定与功能后果,按问题本身安排修复。
核对任务单实现时,将验收条件与代码差异及相关测试逐项对照。查找缺失、只完成一部分或偏离规格的行为,并为每种状态提供证据。不能只凭分支整体看起来已经完成,就代替具体要求之间的比较。
版本审计需要固定发布标签、目标提交和排除范围。检查差异时保持这些参照稳定,让其他审阅者能够复现相同范围。后续尚未发布的修改不混入本次结论,避免移动中的分支静默扩大审阅内容。
仓库修改可能已有清楚的比较点,却没有可供核对的原始规格。按已记录项目标准审阅,并明确说明规格轴已经跳过,不为填补缺口自行编造验收条件。报告应区分哪些结论来自代码约定,哪些需求因材料缺失而无法评估。
审阅范围需要由固定 Git 比较点和原始要求共同确定。收集仓库约定后,分别检查不同审查轴,逐项核实发现,再决定修复和补充测试,避免把印象直接写成缺陷。
提供有效提交、分支或标签,并确认合并基点差异覆盖目标工作。
指出仓库约定与原始需求;缺少需求说明时明确记录。
独立阅读规范和需求发现,不把某一轴通过当作另一轴也合格。
查看其他 Skill 指南,并在使用前核对原始来源。