在另一个用户的pull-request上修改代码的正确方法是什么

Question

方案

如果User X维护了一个回购，并且User A提交了一个提取请求，我希望User B建议在该提取请求之上建立清理或其他功能。

我正在做什么

我这样做的方法是获取User A的repo和功能分支，更新代码并向User X提交一个独立的拉取请求。

问题

但是上面的感觉有点冒险和错误。还有更好的方法吗？

Answer 1

您可以向UserA的功能分支单独提取拉取请求。如果他接受了，你的提交将被添加到原来的请求中。

你也可以在新的请求中引用原始的pull请求，以便它出现在Github的评论中。

如果他不接受拉动请求，则采用douchebaggy方式。他的承诺仍然会出现在他自己的承诺中，因此无关紧要。

Answer 2

最好的方法是根本不使用拉取请求。这就是为什么Git项目不使用pull请求进行开发，而是每个补丁系列都被发送到邮件列表，在那里很容易查看每个补丁，然后原作者修复问题并发送重新滚动的系列，收到更多的评论等。每个补丁系列多次重新滚动并不罕见。

这是Linus Torvalds不喜欢整个GitHub方法的主要原因。

话虽如此，您可以在Github中查看补丁，并请求原作者发送另一个拉取请求。

作为最后的手段，我所做的是我手动将User A遥控器添加到我的本地仓库，获取并自行修复提交，添加适当的注释，或添加新的单独提交，然后合并我的本地分支，而不是远程分支。

Answer 3

2017年更新：

如果提交者允许的话，Github现在allows pushing changes向分支发出了拉取请求。

创建拉取请求时，现在有一个选项：“允许维护者编辑”。默认情况下会进行检查。