Cleanup pre-Gitaly code at Git::DiffCollection
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
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
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.
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?
Edited by 🤖 GitLab Bot 🤖