合并之前先看一眼差异的风险说明
适合谁
这周要交一个仓库的人。面试练习和内容草稿是别的任务。
结果
一份关于缺测试和上下文太薄的风险说明。审查者不会点批准。
做法
- 在工程工位先装 Lauren,再装 PR Reviewer。贴上草稿 PR 或一份 diff。
- 读风险说明。缺测试和缺上下文先于文风挑刺。
- Repo Engineer 可以守仓库上下文。夜间审计是后一步,不是第一次审查。
- 如果例行任务从机器人串到云代理再串到仓库,已发布的失败规则是:告诉你,然后停下。不要盲目重试。
下一步
打开工程工位,先把一份 diff 贴给 PR Reviewer,再添加名单上的其他人。
工程交付台