在 GitHub 上审查拉取请求的推荐方法是什么？

Question

假设一个项目的贡献者提交了一个拉取请求，我需要编译代码以确保它解决了问题。在这种情况下，我通常将作者所做的更改复制到我的本地存储库中，运行所有内容并确保其正常工作，合并 PR，然后运行 ​​git pull 以更新我的本地存储库。

但我刚刚发现 PR 导致了一个新分支。所以我想最好在本地运行 git pull 和 git checkout <branch>，运行代码并确保它解决了问题，然后合并 PR 并删除分支。

一个（非贡献者）fork 一个repo，对他的fork repo 的主分支进行更改，将上游设置为原始repo 并提交PR 的场景呢？这是否会在原始存储库上产生一个新分支，我可以在本地签出该分支以测试提议的更改？

Answer 1

<块引用>

一个（非贡献者）fork 一个repo，对他的fork repo 的主分支进行更改，将上游设置为原始repo 并提交PR 的情况如何？

然后你只是拒绝它并告诉该用户遵循正确的程序：创建一个分支并将该分支作为 PR 提交。不要让一些叉子把你推来推去。

Answer 2

当您是项目的主要主持人时，您可以选择合并其他人提出的拉取请求 因为 PR 包含他们编写的代码 您可以合并或拒绝它

但是当涉及到您的情况时，您可以向维护者以外的其他开发人员添加提及 ('@') 以进行代码审查，这样他们就可以提出一些更改建议，并希望您的 PR 有朝一日能合并