-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
add pyairbyte recipe #26787
base: main
Are you sure you want to change the base?
add pyairbyte recipe #26787
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For airbyte:
For airbyte:
Documentation on acceptable licenses can be found here. |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For airbyte:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipes/airbyte:
Documentation on acceptable licenses can be found here. |
Looks like there are some missing dependencies:
You must create package for those too. |
@ocefpaf you mean I have to create these packages in conda-forge before attempting to merge for pyairbyte? |
Yep. Those are dependencies and that missing. You can add them here, in the same PR. |
I'll get on that now |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/airbyte:
Documentation on acceptable licenses can be found here. For recipes/airbyte-api:
For recipes/airbyte-cdk:
For recipes/airbyte-protocol-models-pdv2:
For recipes/ulid:
For recipes/ulid:
Documentation on acceptable licenses can be found here. |
… ulid meta.yaml files
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/ulid:
For recipes/ulid:
Documentation on acceptable licenses can be found here. |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/python-dotenv:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/airbyte-protocol-models-pdv2:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@ocefpaf I have added all required packages by pyairbyte however, i think this error is from one of the package dependencies and I can't figure it out. Do you mind taking a look? |
Ok I'll do that |
@ocefpaf is it okay to attach a modified local tarball file to my PR for this package? |
You should patch the original source, so we can track the changes. Dumping a modified tarball is not OK. An alternative might be using the GH tag instead. See if they provide one with all the required files for the build. |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/airbyte-protocol-models-pdv2:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Hi @ocefpaf I tried the alternative using gh tag but I am getting a directory error. I also specified a folder but that didn't solve the problem. |
Hi! Thanks for your contribution to conda-forge. If these changes are intentional (and you aren't submitting a recipe), please attach a |
This is something that you should probably ask upstream. I tried to understand their release methods but I could not make heads or tails of it. I did not understand how the sdist tarball that is on PyPI was generated. |
I have made a comment to this effect. Will try again once it is resolved. |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).