pagure

#2197 Remove duplicated code
Merged by pingou. Opened by pingou.
drop_duplication  into  master

Download 2197.patch

We used to have three places doing the diff between multiple branches of
the same or different repo, this is now down to a single one.

This should make it easier to test and fix as needed.

rebased

2 new commits added

  • Increase the unit-tests to test pagure.lib.git.diff_pull_request
  • Rename the test file to reflect more what it tests

rebased

rebased

It would be good to document the arguments and their types here.

LGTM

rebased

rebased

thanks for the review! :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.