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

Approval is optional

Merged by avatar (Jul 26, 2025 5:18am UTC)

Merge details

  • Changes merged into main with f4710808.
  • Deleted the source branch.

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