git - 如何处理审查 pull 请求、修改代码和 merge ？

Question

(100% 浏览器内解决方案:Received a pull-request from someone else, how to edit it before merging directly on github?)

---

假设我在 GitHub 上有一个存储库 myproject。

名为 userblah 的用户提出了一个pull-request。

我尝试了以下，以测试他的修改:

git checkout -b userblah-test    
git pull https://github.com/userblah/myproject.git  

然后我可以在我的文件夹中访问他的版本，没问题。

然后我决定修改他的代码中的一些东西:我在文本编辑器中打开代码，做了一些修改并保存。

然后我想再次切换到my master 分支，但是我报错说我不能切换到master，因为当前已经完成了未提交的修改分支 userblah-test.

正确处理 pull 请求的正确命令是什么？

我的意思是:

1. 将用户代码 pull 入新分支
2. 根据我的喜好稍微修改他的代码
3. 将其推送到我的 master 中，这样用户名 userblah 将被注册为文件的贡献者

Answer 1

我认为您可能希望执行 merge ，这将从一个分支获取更改，并将它们与另一个分支的更改“merge ”( merge )。

假设您想更改建议的代码:

1. git checkout -b userblah-test( checkout 一个名为 userblah-test 的新分支)
2. git pull https://github.com/userblah/myproject.git(从 userblah pull 提议的更改)
3. 做出改变
4. git add .(. 添加所有修改过的文件)
5. git commit(提交更改 - 请务必注意，您仅将更改提交到当前分支，即 userblah-test)
6. git checkout master( merge 前 check out master分支)
7. git merge userblah-test(将在 userblah-test 上所做的所有更改与当前分支 merge )

但是，如果 pull request 是通过 GitHub 提交的，我建议您在 pull request 评论中告知原作者您想要进行的更改。作者应该愿意自己进行这些更改并将它们添加到 pull 请求中。然后，您可以使用 GitHub 自己的 Web 界面 merge 他的 PR - 请参阅 here了解更多信息。

pull 请求旨在促进围绕代码的讨论，如果您对此人提出的更改不是 100% 满意 - 告诉他们!