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

Split bucketfs into Multiple Modules #104

Merged
merged 5 commits into from
Mar 14, 2024

Conversation

Nicoretti
Copy link
Member

The bucketfs package has been restructured to better accommodate future features. The primary change is the modification of __init__.py to serve as the main point for exposing the public interface of the bucketfs package through exasol.bucketfs. All other components have been relocated to private modules, with their public elements made accessible via __init__.py.

Important Notice for Developers:
This reorganization requires careful attention to imports to prevent circular dependencies within the private modules.

@Nicoretti Nicoretti force-pushed the refactoring/refactor-bucketfs-module-structure branch from 44f9641 to 0f4226b Compare March 14, 2024 07:20
Copy link

@ckunki ckunki left a comment

Choose a reason for hiding this comment

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

Is there a changes file describing the changes?
Do we need to notify end users / developers about the changed packages?

@Nicoretti Nicoretti force-pushed the refactoring/refactor-bucketfs-module-structure branch from 0f4226b to 8f19b0f Compare March 14, 2024 11:12
@Nicoretti Nicoretti requested a review from ckunki March 14, 2024 11:12
@Nicoretti Nicoretti force-pushed the refactoring/refactor-bucketfs-module-structure branch from 8f19b0f to c1a2252 Compare March 14, 2024 11:13
@Nicoretti Nicoretti enabled auto-merge (squash) March 14, 2024 11:16
@Nicoretti Nicoretti merged commit 08e4864 into main Mar 14, 2024
17 checks passed
@Nicoretti Nicoretti deleted the refactoring/refactor-bucketfs-module-structure branch March 14, 2024 11:19
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.

2 participants