获得徽章 0
想聊聊一个在编程界经常讨论的话题:拉取请求(Pull Request, PR)到底应该是小巧精致还是大而全呢?
我听到一种观点,认为PR应该越小越好。确实,小PR有它的好处,比如快速审查,易于理解。但我个人有一些不同的看法。👀
首先,我认为对于一些复杂的新功能,一个完整的、较大的PR有时候是必要的。当代码作为一个整体呈现时,审查者可以更全面地理解功能的设计和逻辑,而不是零散地看片段。🧩
其次,小PR可能会导致设计上的小问题逐渐累积到主分支中,因为每个小改动看起来都无害,但累积起来可能就有问题了。而在企业环境中,很少有人会去回过头解决这些问题。😵
再来,审查一个几千行的PR真的那么难吗?我觉得不是。虽然需要投入更多的时间和精力,但审查本就应该是一个认真的过程,而不是随意的几句话就能打发的。
最后,我更倾向于审查那些能够提供完整功能的大型PR,这样我可以更深入地测试和理解它的设计。而不是在小PR中纠结于一些无关紧要的风格细节。
我听到一种观点,认为PR应该越小越好。确实,小PR有它的好处,比如快速审查,易于理解。但我个人有一些不同的看法。👀
首先,我认为对于一些复杂的新功能,一个完整的、较大的PR有时候是必要的。当代码作为一个整体呈现时,审查者可以更全面地理解功能的设计和逻辑,而不是零散地看片段。🧩
其次,小PR可能会导致设计上的小问题逐渐累积到主分支中,因为每个小改动看起来都无害,但累积起来可能就有问题了。而在企业环境中,很少有人会去回过头解决这些问题。😵
再来,审查一个几千行的PR真的那么难吗?我觉得不是。虽然需要投入更多的时间和精力,但审查本就应该是一个认真的过程,而不是随意的几句话就能打发的。
最后,我更倾向于审查那些能够提供完整功能的大型PR,这样我可以更深入地测试和理解它的设计。而不是在小PR中纠结于一些无关紧要的风格细节。
展开
1
2