Remove UpsertExternalServices in favor of internal/db/ExternalServiceStore#Upsert
Created by: asdine
This PR removes UpsertExternalService
from the repo-updater store in favor of internal/db/ExternalServiceStore#Upsert.
Merge request reports
Activity
Created by: sourcegraph-bot
Notifying subscribers in CODENOTIFY files for diff 05caab884b00fd72e1fdd45e94cb02b6c9c9b471...79b33e0553d8405bf4e50d1df2292b063f495908.
Notify File(s) @LawnGnome internal/db/external_services.go @eseliger internal/db/external_services.go @unknwon enterprise/cmd/repo-updater/authz/integration_test.go Created by: codecov[bot]
Codecov Report
Merging #16827 (79b33e0) into main (05caab8) will decrease coverage by
0.05%
. The diff coverage is100.00%
.@@ Coverage Diff @@ ## main #16827 +/- ## ========================================== - Coverage 52.44% 52.39% -0.06% ========================================== Files 1703 1703 Lines 84764 84700 -64 Branches 7615 7615 ========================================== - Hits 44457 44380 -77 - Misses 36406 36420 +14 + Partials 3901 3900 -1
Flag Coverage Δ *Carryforward flag go 51.56% <100.00%> (-0.08%)
integration 29.69% <ø> (ø)
Carriedforward from 05caab8 storybook 28.77% <ø> (ø)
Carriedforward from 05caab8 typescript 54.38% <ø> (ø)
Carriedforward from 05caab8 unit 34.99% <ø> (ø)
Carriedforward from 05caab8 *This pull request uses carry forward flags. Click here to find out more.
Impacted Files Coverage Δ internal/db/external_services.go 66.23% <ø> (ø)
internal/repos/observability.go 92.80% <ø> (-0.33%)
internal/repos/store.go 79.10% <ø> (-4.05%)
cmd/repo-updater/repoupdater/server.go 57.88% <100.00%> (ø)
internal/repos/syncer.go 77.10% <100.00%> (ø)
.../internal/codeintel/resolvers/graphql/locations.go 83.50% <0.00%> (-2.07%)
Please register or sign in to reply