Skip to content

Remove unused code from DiffTree

Vasilii Iakliushin requested to merge 143216_remove_unused_code into master

What does this MR do and why?

Originally, DiffTree was introduced to handle a special case for initial commit diffs. Gitaly didn't return a diff of the original commit when it was provided to FindChangedPaths RPC.

The fix Fix push rules validation on initial push (!123950 - merged) solved this problem. It allowed to provide a DiffTree instance to the RPC and use a tree interface to fetch changes even for the initial commit.

Later, a better option was discovered: Refactor FindChangedPaths RPC call for initial ... (!143216 - merged).

As a result, some old code is not needed anymore.

Merge request reports