- 检查范围:是否在项目接受范围内,避免白做
- 引用贡献指南(CONTRIBUTING):给出规则链接而不是口头习惯
- 必须项清单化:测试、文档、变更日志各自明确
- 回应时肯定投入:首次贡献者的体验决定他是否回来
外部贡献者看不到内部上下文:评审描述要自包含。
| 前置对齐 | 先开 issue 讨论方案与范围,避免做完被拒 |
|---|---|
| 必查项 | 测试、文档、变更日志与贡献者许可流程 |
开源项目评审外部 PR:先对齐贡献指南与范围,再评审实现;对首次贡献者友好表达,明确列出必须修改项与后续步骤。
| 前置对齐 | 先开 issue 讨论方案与范围,避免做完被拒 |
|---|---|
| 必查项 | 测试、文档、变更日志与贡献者许可流程 |