sul-dlss / dlme-airflow

This is a new repository to capture the work related to the DLME ETL Pipeline and establish airflow
Apache License 2.0
1 stars 0 forks source link

Post harvest dry refactor #477

Closed aaron-collier closed 5 months ago

aaron-collier commented 5 months ago

@jacobthill I think this is ready to merge into your branch. From there I will review your branch and get this into main.

This looks like a HUGE pr... it's mostly tests and test fixture contributing the number of files and number of lines added. I may pare that big fixture down to improve the test a bit, but I can do that as a follow up.

That said, it may be worth walking through this as there is the change to the configs.