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

【问题标题】：What is the recommended way to review pull requests on GitHub?在 GitHub 上审查拉取请求的推荐方法是什么？
【发布时间】：2021-04-12 12:23:06
【问题描述】：

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

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

如果（非贡献者）分叉一个 repo，更改他的分叉 repo 的主分支，将上游设置为原始 repo，然后提交 PR，情况会怎样？这是否会在原始存储库上产生一个新分支，我可以在本地签出以测试提议的更改？

【问题讨论】：

标签： git github

【解决方案1】：

如果（非贡献者）分叉一个 repo，对他分叉的 repo 的主分支进行更改，将上游设置为原始 repo，然后提交 PR，情况会怎样？

然后你拒绝它并告诉该用户遵循正确的程序：创建一个分支并将该分支作为 PR 提交。不要让一些 forker 左右你。

【讨论】：

【解决方案2】：

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

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

【讨论】：