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
fix(ingest): fix metadata for custom python packages #9391
Conversation
Also moves the _Aspect type to a common location to prevent it from being defined twice in the eyes of type checkers.
This package should be installed alongside the main acryl-datahub package. | ||
|
||
Install the custom package locally with `pip install custom-package/my-company-datahub-models` | ||
To enable others to use it, share the file at custom-package/my-company-datahub-models/dist/*.whl and have them install it with `pip install <wheel file>.whl` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The star makes me think there's multiple such files, but I think there should just be one? Or do we have multiple for multiple python versions, or something like that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes there should be just one file - I'll make it a bit more clear
package_data={{ | ||
"{python_package_name}": ["py.typed"], | ||
"{python_package_name}.models": ["schema.avsc"], | ||
"{python_package_name}.models.schemas": ["*.avsc"], | ||
}}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's this do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
by default it only packages .py files, so this tells it to also include the raw schema files in the .whl and .tar.gz files
Also moves the _Aspect type to a common location to prevent it from being defined twice in the eyes of type checkers.
Checklist