Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

GitHub does not recognize changes from a reverted pull request

I'm working on a Git branch called bug-fix-1, which I created based off integration branch. There was another branch bug-fix-2 which was merged into integration branch and then we reverted the changes (all were done on Github).

Now when I put up a new pull request to merge my bug-fix-1 into integration, I expect to see the changes which I pulled from bug-fix-2. But no, I can't see any of them, the pull request only show the changes I made on bug-fix-1.

Any solution on how do I pull in the changes from bug-fix-2 into my branch and merge them together to integration ?

like image 891
Dio Phung Avatar asked Jan 07 '17 01:01

Dio Phung


1 Answers

This is an interesting feature of Git, and not such a bad one once you realize what is happening.

Git tracks commits by commit hash, and from the history hot-fix-2 is already present. Git also knows that the reversion has been applied, but it doesn't really see it as backing out the original commit. Instead, it's just another commit that happens to apply the same diff in reverse, and it has it's own commit hash too.

In order to allow Git to see the changes, you have one of several choices:

  1. Revert the revert. As weird as it sounds, it works. This says to introduce a new commit that contains the reverse diff. That commit is not present on the integration branch, and so Git will see it as a change to be brought in.

  2. You can cherry pick the original commit, but you'll need a special option:

    git cherry-pick --keep-redundant-commits COMMIT_HASH
    

    where COMMIT_HASH introduces the original bug fix.

There are other ways to help do this too, but they're a bit more complicated and involve using git rebase. I think the above techniques are probably the most straight-forward.

like image 143
John Szakmeister Avatar answered Oct 19 '22 15:10

John Szakmeister