Skip to content

refactor: merge catalog crawler related core modules#5762

Merged
ndr-brt merged 1 commit into
mainfrom
5744-merge-crawler-core-modules
May 21, 2026
Merged

refactor: merge catalog crawler related core modules#5762
ndr-brt merged 1 commit into
mainfrom
5744-merge-crawler-core-modules

Conversation

@ndr-brt

@ndr-brt ndr-brt commented May 20, 2026

Copy link
Copy Markdown
Member

What this PR changes/adds

Merges catalog crawler related core modules into a single catalog-crawler module

Why it does that

simplicity, module reduction

Further notes

  • moved transformers into dsp related modules, as that's their place
  • I noticed different modules used in tests to expand json-ld, there could be a single fixture offered by the json-ld module, issue will come

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Closes #5744

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@ndr-brt ndr-brt requested a review from a team as a code owner May 20, 2026 06:43
@ndr-brt ndr-brt added the refactoring Cleaning up code and dependencies label May 20, 2026
@ndr-brt ndr-brt force-pushed the 5744-merge-crawler-core-modules branch 4 times, most recently from d2c1ba4 to 34b8df3 Compare May 20, 2026 16:12
@ndr-brt ndr-brt force-pushed the 5744-merge-crawler-core-modules branch from 34b8df3 to cdf9a9e Compare May 21, 2026 06:54
@ndr-brt ndr-brt merged commit 9c440cb into main May 21, 2026
34 of 35 checks passed
@ndr-brt ndr-brt deleted the 5744-merge-crawler-core-modules branch May 21, 2026 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Cleaning up code and dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Merge crawler-core and federated-catalog-core modules

2 participants