The Theatre of Pull Requests and Code Review
8 months ago
- #pull requests
- #code review
- #git
- 萨沙·尤里奇在Goatmire Elixir大会上的演讲《给我讲个故事》将戏剧化叙事与代码审查及PR的技术建议相结合
- 代码审查常见问题包括庞大复杂的PR难以理解和测试，导致流于表面的'LGTM'评论
- 萨沙主张将难以理解的PR退回作者，专注于5-10分钟可完成审查的PR
- 理想的PR应范围小（建议300行代码以内），提交记录能讲述连贯故事
- 故事化的提交信息帮助审查者理解修改背景和思维过程
- 通过迭代编写和优化提交（包括使用fixup提交）来保持清晰可追溯的提交历史
- 确保每个提交都可编译且应用可运行，这对调试和使用git bisect等工具至关重要
- 具有清晰提交故事的聚焦PR能获得更好反馈，加快开发周期并提升代码质量