PR#2129: Remove mentioning if a commit is in master on the front page -

pagure

#2129 Remove mentioning if a commit is in master on the front page

Merged Mar 20, 2017 by pingou. Opened Mar 20, 2017 by pingou.

drop_difflist into master

pingou commented Mar 20, 2017

This had a few problems:
- We were comparing master with master of the parent repo, without taking
into account projects not using master for their main branch
- We were comparing master with master of the parent repo, but for regular
project (ie not fork) we were thus comparing it to itself
- The commits present in the repo and not its parent are shown on the
bottom of the column on the right, while we already show in the list
of branches which one is in a PR and which one could be.

So overall, we can just drop this functionality

puiterwijk commented Mar 20, 2017

This drops a lot of walks and steps, so +1 from performance perspective.
The change itself also looks reasonable to me.

pingou commented Mar 20, 2017

Thanks! :)

Pull-Request has been merged by pingou

Mar 20, 2017