<!--IssueSummary start--> <details> <summary> Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards. </summary> - [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=30419) </details> <!--IssueSummary end--> After some investigation at https://gitlab.com/gitlab-org/gitlab-ce/issues/65013#note_195533872 I've noticed that we got some ~~unused code at [`Gitlab::Git::DiffCollection`](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/lib/gitlab/git/diff_collection.rb#L51) which was likely not deleted when we migrated to Gitaly.~~ It handles a bit of collapsing / limitation logic, even though most of it (if not all) is already being done [at Gitaly](https://gitlab.com/gitlab-org/gitaly/blob/master/internal/diff/diff.go#L171-190). We could also rename `each_gitaly_patch` to just `each_patch`. Given you've worked at it at the time, do you see any reason for keeping that code @jacobvosmaer-gitlab?