Skip to content
Snippets Groups Projects

remove types.Event and replace with database.Event

Merged Warren Gifford requested to merge usage-data/refactor-event-types into main

Created by: coury-clark

This PR refactors all usages of types.Event to instead use database.Event. Reason being is it was very strange that the EventLogStore insert operations take in a database.Event but the read operations return a types.Event. I need to make some updates here, and since there is already significant overlap I just refactored them all to use database.Event.

Test plan

I started Sourcegraph and ensured events can insert, read, and pings can be created correctly. Also looking for CI passing for some additional confidence here.

Merge request reports

Merged by avatar (Jul 12, 2025 9:25pm UTC)

Loading

Activity

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