关键是要注意代码以检查它是否正确"。如何在不接受并运行代码的情况下知道代码是否实际运行并且不会失败。 (如果它错了,那就还原吧)这似乎有些令人费解。测试是否应该由发起拉取请求的人负责?或者是否有一些工作流程可以在将代码接受为主代码之前运行代码?
答案 0 :(得分:1)
许多GitHub项目使用Travis (or other CI) integration在所有分支上运行构建,包括拉取请求。所以pull请求只是一个很好的git集成方式来向项目提交补丁。我将其视为带有补丁的#34;错误报告"选项,与简单的错误报告"。
相比答案 1 :(得分:0)
审核代码
在处理项目时,您希望确保团队中的每个人都知道代码库中发生了什么或从代码库中删除了什么。此外,第2或第3 ......或第4或第5组眼睛不会受伤。
测试代码
CI服务(la TravisCI)将自动运行以确保没有任何损坏,并且是确保绿色主人构建的第一道防线 - 我不能强调这一步。
讨论代码
此功能/请求是否应该是代码库的一部分?有没有更好的方法来实现它?
检查点
当前的部署是否已损坏?找到合并提交/拉取请求节点而不是单个提交节点要容易得多。
通信
围绕代码永远不会有足够的沟通。有人总是在循环之外