执行git pull --rebase upstream master为何污染GitHub PR diff？

我来帮你捋捋实际用Git时容易踩的几个坑，这大概率是其中某一个导致的：

• 冲突解决时手滑带了别人的代码：rebase过程中遇到冲突时，手动合并代码很容易出错——尤其是冲突块比较复杂的时候，一不小心就会把别人提交里的代码留在文件里，甚至直接提交了这些内容。解决冲突时一定要仔细对比两边差异，只保留自己的修改。
• 本地分支藏了不属于你的提交：说不定你之前不小心合并了别人的分支，或者Cherry-pick了别人的提交但没清理干净。这些“外来”提交在rebase之后依然会留在你的分支上，自然会出现在PR的diff里。你可以用git log --oneline看看提交历史，一眼就能认出哪些不是你自己的提交。
• PR的比较基准搞错了：虽然你说PR基准是upstream/master，但有时候平台（比如GitHub）可能会自动调整，或者你创建PR后不小心切换了比较目标到自己fork的master——如果你的fork master没同步upstream的最新代码，那diff里就会包含upstream和你fork之间的所有差异。赶紧去PR页面确认一下，比较的是不是upstream:master和你的特性分支。
• 远程仓库配置出问题了：先检查下你的upstream远程仓库是不是正确的上游地址，用git remote -v就能看。如果upstream指向的是你自己的fork，那git pull --rebase upstream master拉的就是你自己fork的代码，自然会和真正的上游master有差异。要是错了，用git remote set-url upstream <正确的上游仓库URL>改过来就行。

对应的解决办法： #

• 排查提交历史：用git log --oneline找出不属于你的提交，然后执行git rebase -i upstream/master进入交互式变基，把那些多余的提交删掉。
• 重新变基并仔细处理冲突：先执行git rebase --abort回到变基前的状态，再重新跑git pull --rebase upstream master，解决冲突时一定要慢，别着急提交。
• 确认PR基准：去PR页面手动调整比较基准为upstream:master，确保diff是和上游最新代码的对比。
• 修正远程仓库配置：如果upstream地址错了，赶紧改过来，再重新拉取变基。

内容的提问来源于stack exchange，提问作者Heisenberg