eladkal commented on code in PR #25930: URL: https://github.com/apache/airflow/pull/25930#discussion_r956038963
########## setup.py: ########## @@ -599,12 +599,21 @@ def get_all_db_dependencies() -> List[str]: # to separately add providers dependencies - they have been already added as 'providers' extras above _all_dependencies = get_unique_dependency_list(EXTRAS_DEPENDENCIES.values()) +_all_dependencies_without_airflow_providers = list( + filter(lambda k: 'apache-airflow-' not in k, _all_dependencies) +) + # All user extras here -EXTRAS_DEPENDENCIES["all"] = _all_dependencies +# all is purely development extra and it should contain only direct dependencies of Airflow +# It should contain all dependencies of airflow and dependencies of all community providers, +# but not the providers themselves +EXTRAS_DEPENDENCIES["all"] = _all_dependencies_without_airflow_providers # This can be simplified to devel_hadoop + _all_dependencies due to inclusions # but we keep it for explicit sake. We are de-duplicating it anyway. -devel_all = get_unique_dependency_list([_all_dependencies, doc, devel, devel_hadoop]) +devel_all = get_unique_dependency_list( + [_all_dependencies_without_airflow_providers, doc, devel, devel_hadoop] +) Review Comment: Great. Everything should be green now -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
