- 评论带背景与参考:规则、文档链接、历史决策
- 轮换评审人:避免只有少数资深成员掌握全貌
- 新人先从评审测试与文档改动开始,逐步加深
- 作者在描述里讲清设计:读者顺便获得领域知识
带背景的评审评论三段式
带背景的评审评论三段式
结论:这处建议从参数传入,而不是读全局变量。
依据:该服务并发处理请求,全局变量会互相覆盖(正确性问题)。
参考:团队规范「状态管理」一节,src/cache.py 有同类写法可沿用。
要点:评论给出规则出处与既有实现,让作者学到团队约定而非只听结论。知识传递动作
| 场景 | 动作 |
|---|---|
| 新人首次评审 | 配对资深评审人共同完成一轮,事后对照关注点复盘 |
| 跨模块改动 | 邀请目标模块维护者参与,顺带补齐模块认知 |
| 重复出现的评论 | 写进模块文档或评审清单,减少口头重复与单点依赖 |
评审的隐藏产出是集体知识:每个 PR 都是团队共享上下文的载体。