Skip to content

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 🤖