potiuk commented on a change in pull request #15781:
URL: https://github.com/apache/airflow/pull/15781#discussion_r632445771
##########
File path: setup.py
##########
@@ -524,15 +532,15 @@ def get_sphinx_theme_version() -> str:
# Dict of all providers which are part of the Apache Airflow repository
together with their requirements
PROVIDERS_REQUIREMENTS: Dict[str, List[str]] = {
- 'airbyte': [],
+ 'airbyte': http,
Review comment:
Not really. Those are optional dependencies (in extras). Cross-provider
dependencies are extra/optional ones (because ofen those are optional
dependencies of only one operator or so (like in case of slack). Before that,
when http was mandatory, it did not matter. But now we have to explicitly
change it to 'apache-airflow-provider-http` actuatlly if we want to be 100%
correct.
--
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]
