Skip to content

Refactoring transformation and load - #586

Merged
palkerecsenyi merged 9 commits into
CERNDocumentServer:masterfrom
kpsherva:refactoring-transformation
Aug 31, 2026
Merged

Refactoring transformation and load#586
palkerecsenyi merged 9 commits into
CERNDocumentServer:masterfrom
kpsherva:refactoring-transformation

Conversation

@kpsherva

Copy link
Copy Markdown
Contributor

@kpsherva
kpsherva force-pushed the refactoring-transformation branch 3 times, most recently from dc4f052 to 6f8eeb2 Compare August 25, 2026 15:52
- return typed dictionaries to help with data flow
- avoid unnnecessary inheritance
- separate data checking from data building (Entry vs. Transform class)

# Conflicts:
#	cds_migrator_kit/rdm/records/load/load.py
#	cds_migrator_kit/rdm/records/transform/transform.py
- encapsulate data keys handling within specific entity
- isolate responsibilities of transforming and validation
- remove data layer access from rules transformation
* refactor code to specialise on responsibilities
@kpsherva
kpsherva force-pushed the refactoring-transformation branch from 6f8eeb2 to 76b688e Compare August 26, 2026 12:25
@palkerecsenyi
palkerecsenyi force-pushed the refactoring-transformation branch from 76b688e to 3f60b4a Compare August 28, 2026 11:22
@palkerecsenyi
palkerecsenyi merged commit f9f4744 into CERNDocumentServer:master Aug 31, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

merge EP and regular workflow

2 participants