-
-
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 airflow 1.8.0 recipe to conda-forge. #2882
Conversation
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 ( |
This is a requirement for the airflow PR conda-forge#2882
This is a requirement for the airflow PR conda-forge#2882
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 ( |
This is a requirement for the airflow PR conda-forge#2882
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 just wanted to let you know that I linted all conda-recipes in your PR ( |
This is a requirement for the airflow PR conda-forge#2882
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 ( |
This is the initial list that must be discussed - https://gist.github.com/sodre/a5fb32df81d1aabc7383565aecc29c84
@conda-forge/core, One possibility is to create several airflow- metapackages that brings in the additional dependencies. Is there a conda-forge example for that? |
So for now i would try and get a minimal airflow recipe building fully on conda-forge. Treating the additional dependencies as metapackages for now is probably the simplest approach. For a few of those sets of dependencies you are going to have to submit additional conda-forge packages in any case so it may be simpler for you to iterate that way? |
Have heard |
In the short term I will go with @mariusvniekerk suggestion of a minimal airflow install. I will create an issue for the missing extra-packages for airflow and slowly create the associated metapackages. The new feature in for handling optional dependencies in conda through meta.yaml is solved through conda/conda-build#2001. Once it becomes part of conda-forge we can augment the main recipe. |
@conda-forge/core this is ready for review. |
Congrats! |
I need to start getting the add-ons done, or just wait for conda 4.4. |
The recipe currently does not build pass the test because
I will be working to create additional packages for those.