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

Break into subpackages #77

Closed
wants to merge 2 commits into from

Conversation

jaybythebay
Copy link
Contributor

@jaybythebay jaybythebay commented May 2, 2024

@JustinGrilli Here's a rough start to fix the issue of package installation failing on Apple Silicon computers outlined #75

The approach is to separate the code into 2 namespaced packages. I think this has a few advantages:

  1. Keeps all the code within packages that can be used outside of Airflow
  2. Cleaner for maintaining the functions and code

Check out these docs for a brief overview of naming spacing if you're not familiar. It's new to me!

Let me know if you like this approach as I would like architectural agreement if before I do more work.

@jaybythebay jaybythebay marked this pull request as draft May 2, 2024 07:08
@jaybythebay jaybythebay closed this May 3, 2024
@jaybythebay jaybythebay deleted the break-into-subpackages branch May 4, 2024 14:27
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.

1 participant