potiuk commented on a change in pull request #11487:
URL: https://github.com/apache/airflow/pull/11487#discussion_r503864962
##########
File path: setup.py
##########
@@ -749,14 +750,18 @@ def is_package_excluded(package: str, exclusion_list:
List[str]):
def do_setup():
"""Perform the Airflow package setup."""
write_version()
+ # Note! Exclude does not work when combined with 'airflow' include so we
need to remove
+ # providers manually
+ all_packages = [package for package in find_packages(include=['airflow',
'airflow.*'])
+ if not package.startswith('airflow.providers')]
setup(
name='apache-airflow',
description='Programmatically author, schedule and monitor data
pipelines',
long_description=long_description,
long_description_content_type='text/markdown',
license='Apache License 2.0',
version=version,
- packages=find_packages(include=['airflow', 'airflow.*']),
+ packages=all_packages,
Review comment:
BTW. I stil have to test all the behaviour in Dockerfile/local
installation with this setup.py - i.e. how people will be testing it locally. I
believe all works fine for pip install -e . and when I add extras -> providers
links it will work as well for the Dockerfile But this can be done later.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
