Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor S3 components into common module for reuse #995

Merged
merged 3 commits into from
Nov 7, 2023

Conversation

davidsloan
Copy link
Collaborator

Remaining transformers into common. Unit testing. Removing scala testcontainers lib from S3 and reusing the test-common S3 test containers Extract shared code to cloud-common
Renaming some common things which shouldn't have S3 in their name GCP Config WTD
Clean up, reduce tech debt, remove duplicated functionality, avoid weak string referencing Updating references to S3 in common module

Remaining transformers into common.  Unit testing.
Removing scala testcontainers lib from S3 and reusing the test-common S3 test containers
Extract shared code to cloud-common
Renaming some common things which shouldn't have S3 in their name
GCP Config WTD
Clean up, reduce tech debt, remove duplicated functionality, avoid weak string referencing
Updating references to S3 in common module
@davidsloan davidsloan changed the title Unit testing Refactor S3 components into common module for reuse Oct 25, 2023
@davidsloan davidsloan marked this pull request as ready for review October 25, 2023 14:54
Copy link

@FPerezP FPerezP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my limited knowledge of this code base, everything looks good. I would probably wait for Stefan to confirm but other than that, feel free to merge from my side. Thanks a lot for this refactoring and reorganization! Looks nice and necessary!

@davidsloan davidsloan merged commit 5b598c4 into master Nov 7, 2023
144 checks passed
@davidsloan davidsloan deleted the feat/split-common-module branch November 7, 2023 15:28
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.

3 participants