kaxil commented on a change in pull request #11526:
URL: https://github.com/apache/airflow/pull/11526#discussion_r510073225
##########
File path: setup.py
##########
@@ -741,13 +839,27 @@ def is_package_excluded(package: str, exclusion_list:
List[str]):
]
+def get_provider_package_from_package_id(package_id: str):
+ """
+ Builds the name of provider package out of the package id provided/
+
+ :param package_id: id of the package (like amazon or microsoft.azure)
+ :return: full name of package in PyPI
+ """
+ package_suffix = package_id.replace(".", "-")
+ return f"apache-airflow-providers-{package_suffix}"
+
+
def do_setup():
"""Perform the Airflow package setup."""
- install_providers_from_sources =
os.getenv('INSTALL_PROVIDERS_FROM_SOURCES')
- exclude_patterns = \
- [] if install_providers_from_sources and
install_providers_from_sources == 'true' \
- else ['airflow.providers', 'airflow.providers.*']
+ install_providers_from_sources =
os.getenv('INSTALL_PROVIDERS_FROM_SOURCES') == 'true' or False
Review comment:
```suggestion
install_providers_from_sources =
bool(os.getenv('INSTALL_PROVIDERS_FROM_SOURCES') == 'true')
```
----------------------------------------------------------------
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]
