Skip to content
Snippets Groups Projects

campaigns: Cleanup some little things

Merged Warren Gifford requested to merge es/syncer-experiments into main

Created by: eseliger

I found these while browsing the code, trying to understand the selection of auth tokens for the syncer.

  • The repoStore.Get method already returns an error with that error message, so no need to wrap it another time. Saves us a copy-pasted helper
  • Handle potential nil index access in buildChangesetSource
  • Correctly store error in syncChangeset so transactions are correctly rolled back

Merge request reports

Approval is optional

Merged by avatar (Jul 13, 2025 6:54am 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