pagure

#1017 Do not render the diffs of large added and removed files
Merged by ryanlerch. Opened by ryanlerch.
ryanlerch/pagure PR-difflist  into  master

Download 1017.patch

Rendering large diffs in PRs is really slow. To counter this
this commit does not render diffs for added and removed files
that are over 100 lines, instead it provides a link to view the file.

In the case of a removed file, it provides a link to the file as it
was the before the commit.

100 lines is still fairly short, could we say 1000 and see if that helps?

One comment, otherwise :thumbsup:

rebased

Upped the limit to 1000, there was no noticible change with my test set, we should keep an eye on larger PRs in prod though once this goes out.

Pull-Request has been merged by ryanlerch
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.