主要观点:这是关于拉取请求审查的一般解释,认为无关更改应拆分为单独的 PR,较大的拉取请求审查和合并时间长、成本高、风险大,应尽量减少其成本,如按每个任务一个 PR、分离逻辑和样式更改等,同时也指出这些不是绝对规则,有时包含小的无关更改更划算,还推荐了进一步阅读的内容。
关键信息:
- 解释了为什么要将无关更改从审查中分离出来,包括减少审查时间和成本等。
- 提出按每个任务一个 PR 的策略,使审查更简单,且便于生成变更日志和发布。
- 区分了逻辑更改和样式更改,强调分离它们可提高审查效率,同时可通过自动化工具处理样式偏好。
- 说明这些不是绝对规则,有时可包含小的无关更改并添加注释解释。
重要细节: - 以著名作家塞缪尔·约翰逊的画像为例,说明拉取请求的相关情况。
- 举例对比了包含逻辑和样式更改的代码与仅含逻辑更改的代码,展示了分离的优势。
- 推荐了一些关于小拉取请求的进一步阅读内容,如谷歌测试博客和软件工程栈交换的相关问题。
**粗体** _斜体_ [链接](http://example.com) `代码` - 列表 > 引用
。你还可以使用@
来通知其他用户。