Skip to content

Search backend: simplify searcher job

Warren Gifford requested to merge backend-integration/cc/simplify into main

Created by: camdencheek

Two small refactor commits in this PR:

  1. Remove a temporary construction of search.RepositoryRevisions. This was awkward because we're constructing it just to deconstruct it when used. This led to some confusing stuff like unchecked repoRevs.Revs()[0], which is really confusing for passers-by.
  2. Stop nesting g.Go(). We're spinning up a goroutine only to join it immediately after, which is confusing.

Test plan

Backend integration tests. Semantics-preserving.

Merge request reports

Loading