Skip to content
Snippets Groups Projects

DB Backend: remove `basestore.NewWithDB`

Merged Warren Gifford requested to merge cc/remove-new-with-db-2 into main

Created by: camdencheek

This replaces the last instances of basestore.NewWithDB() and removes the function. Instead of constructing stores from a dbutil.DB interface, now we construct stores using other shareable handles or from transactable handles directly.

Depends on https://github.com/sourcegraph/sourcegraph/pull/36924 (sorry for the weird dependency. I had a couple of branches going. Will rebase once that's merged)

Test plan

Unit tests.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
Please register or sign in to reply
Loading