You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by ka...@apache.org on 2021/08/13 09:15:45 UTC

[airflow] branch main updated: Remove redundant note on HTTP Provider (#17595)

This is an automated email from the ASF dual-hosted git repository.

kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new f2a8a73  Remove redundant note on HTTP Provider (#17595)
f2a8a73 is described below

commit f2a8a73ca99912d25e9cd8142038cf3a6df622d6
Author: Kaxil Naik <ka...@gmail.com>
AuthorDate: Fri Aug 13 10:15:30 2021 +0100

    Remove redundant note on HTTP Provider (#17595)
    
    Since https://github.com/apache/airflow/pull/16974 we have switched back to including HTTP provider and this comment now is not correct.
---
 setup.py | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/setup.py b/setup.py
index 0fc6da8..75f87ca 100644
--- a/setup.py
+++ b/setup.py
@@ -340,14 +340,6 @@ http = [
     'requests>=2.26.0',
 ]
 http_provider = [
-    # NOTE ! The HTTP provider is NOT preinstalled by default when Airflow is installed - because it
-    #        depends on `requests` library and until `chardet` is mandatory dependency of `requests`
-    #        See https://github.com/psf/requests/pull/5797
-    #        This means that providers that depend on Http and cannot work without it, have to have
-    #        explicit dependency on `apache-airflow-providers-http` which needs to be pulled in for them.
-    #        Other cross-provider-dependencies are optional (usually cross-provider dependencies only enable
-    #        some features of providers and majority of those providers works). They result with an extra,
-    #        not with the `install-requires` dependency.
     'apache-airflow-providers-http',
 ]
 jdbc = [