Skip to content
Snippets Groups Projects

Refactor out sources logic into separate package

Merged Administrator requested to merge es/sources-pkg into main

Created by: eseliger

We had syncer, reconciler executor and the service all three require a changeset source, with some sort of authentication in place. The logic for this was different in all three places, and I was about to introduce a 4th place where I'd use it - so I figured this is a good moment to do a nice refactor and have a single source of truth for external service selection, authenticator usage and clone URL computation. Say hello to the new sources package. We might be able to wrap existing GitHubSource,... and such in the future so we can pull the ChangesetSource logic into our codebase ... let see, stay tuned. For now, this is enabling us to easily get an authenticated ChangesetSource for a given changeset or repo.

Merge request reports

Merged by avatar (Jul 7, 2025 8:29pm 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
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading