GPT中文小秘书挑几下就能开工
我积分

把 PR 先过一遍出评审意见

关于「把 PR 先过一遍出评审意见」:能拿到什么、要点选什么、怎么办

大 PR 逐文件挑问题,评审意见一次列清。PR 代码评审是逐文件挑出逻辑、边界、安全、规范问题并分级;在 GPT中文网点选几项,能直接拿到评审意见表和打回建议。

能拿到什么

  • 推荐:PR 评审意见表(xlsx)(列:文件/行、问题描述、级别、建议改法、checklist命中项;附汇总)

要你点选的几项

变更类型、技术栈、上传 PR diff、上传评审清单、上传扫描报告、评审严格度、重点检查项、含级别统计与文件分布。多数点一下就行,没选的按常见情况替你定,交付时说清哪几项是按默认的。

怎么办

  1. 上传 PR diff 和评审清单
  2. 选严格度和重点检查项
  3. AI 逐文件列出问题并分级
  4. 核对误报并定打回结论
  5. 导出评审意见表和勾选表

别人还这样说

  • 帮我看看这个 PR 的 diff,列一下评审意见,按严重程度分个级。
  • 这是 checklist 和扫描报告,PR 太大了,帮我逐文件挑问题,给个打回结论。
直接说你要办的事。下面是别人怎么办的,点任何一项就换成你自己的。左边是别人怎么办的,右边是你的,点一项就开始改。
示范 · 别人这样说

「把这份代码变更过一遍,边界和安全方面的问题单独标出来,我需要一份评审表。」

替他选了:逻辑/边界

按示范选好的 点一项就变成你的

你的成品会在这里出现
示例成品 · 不是你的按示范写的,看个样子
示例:PR 评审意见表(示意) 文件/行 | 问题 | 级别 | 建议改法 src/api/user.js:45 | 用户 ID 未做空值判断,可能导致 500 | 高 | 加 null 判断并返回 400 src/service/order.js:112 | 订单金额计算使用浮点数直接相加 | 高 | 改用整数分或 decimal 计算 src/utils/format.js:8 | 函数缺少 JSDoc 注释 | 低 | 补齐参数和返回值说明 src/components/List.vue:33 | 列表 key 使用 index,重排可能错乱 | 中 | 改用唯一 id 作 key src/config/db.js:5 | 数据库密码硬编码在源码 | 高 | 迁移到环境变量或密钥管理 汇总:高 3 条,中 1 条,低 1 条;文件分布:src/api 1 条,src/service 1 条,src/utils 1 条,src/components 1 条,src/config 1 条。 检查项勾选表(示意): 检查项 | 是否通过 空值和边界处理 | 未通过 敏感信息不硬编码 | 未通过 命名和注释规范 | 部分通过 单元测试覆盖新增逻辑 | 未通过 结论建议:存在高优先级问题,建议打回修改后重新提交。(最终是否打回由评审人决定)

GPT中文 也可能出错,成品请自己核对关键数字和事实。条款 · 隐私