tree 98aa2cb6a7c54afe44eb5034057a27cde228fb87
parent 8b20dc7a565d86cb36f7776dfb7619d6d6d8aaed
author Tatiana Bradley <tatianabradley@google.com> 1728659492 -0400
committer Tatiana Bradley <tatianabradley@google.com> 1729533085 +0000

internal: unify docs Sync

Refactor the internal packages to re-use code for syncing data
to a [docs.Corpus].

The *docs packages were all essentially doing the same thing, requiring
us to create a new package and copy code every time we wanted to add
a new data source.

With this change, a data source that implements the [docs.Source]
interface (and whose events implement the [docs.Embeddable] interface)
can call [docs.Sync] to sync their new events to a corpus.

This change allows us to delete the githubdocs, discussiondocs,
gerritdocs and crawldocs packages. All tests are retained and moved to
the client packages.

Note also that the Sync functions had some unused parameters and
unused return values (error), which are cleaned up here.

Change-Id: I7fb5b360d7f7409fc7cec91c70fce4519a122067
Reviewed-on: https://go-review.googlesource.com/c/oscar/+/619697
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Zvonimir Pavlinovic <zpavlinovic@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>