ashb commented on a change in pull request #15781:
URL: https://github.com/apache/airflow/pull/15781#discussion_r632423430



##########
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:
       Same for airbyte, apache.livy, and opsgenie as for slack -- we should 
depend upon the http provider, not http deps directly.
   
   I can't remember where we define cross-provider deps though.




-- 
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]


Reply via email to