[email protected] wants to merge refs/heads/ojarjur/fix-request-bug into refs/heads/master
2015-12-17 01:32:44

Fix a bug where requesting a review would fail with an erroneous message.

We were figuring out the set of commits to include in a review by listing the commits between the head of the target ref and the head of the source ref. However, this only works if the source ref is a fast-forward of the target ref.

This commit changes it so that we use the merge-base of the target and source refs as the starting point instead of the target ref.

ckerur commented on 2015-12-17 23:09:19 with status 👍