git: Command retrying made a copy that was never used (!11807) · Merge requests · Administrator / sourcegraph

Administrator requested to merge double-ensure-revision into master Jun 30, 2020

Created by: ryanslade

commandRetryer is supposed to perform its first attempt without trying to resolve missing revisions. The previous code made a copy with removed fields to do this, but didn't actually use the copy.

In practice this means that when a command was run against a missing revisions with a supplied value in the EnsureRevisions field we would actually try to fetch it twice from the remote.

Part of https://github.com/sourcegraph/sourcegraph/issues/11654

git: Command retrying made a copy that was never used

Merge request reports