You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/01/14 16:30:07 UTC

[GitHub] [airflow] kaxil commented on a change in pull request #20878: Unpin ``argcomplete`` and ``colorlog``

kaxil commented on a change in pull request #20878:
URL: https://github.com/apache/airflow/pull/20878#discussion_r784984948



##########
File path: setup.cfg
##########
@@ -81,14 +81,14 @@ setup_requires =
 #####################################################################################################
 install_requires =
     alembic>=1.5.1, <2.0
-    argcomplete~=1.10
+    argcomplete>=1.10
     attrs>=20.0, <21.0
     blinker
     cached_property~=1.5;python_version<="3.7"
     cattrs~=1.1, !=1.7.*
     # Required by vendored-in connexion
     clickclick>=1.2
-    colorlog>=4.0.2, <6.0
+    colorlog>=4.0.2

Review comment:
       No strong opinion, I went back and forth with it before creating the PR but remove those as we recommend using constraints for installation anyway so keep the deps "open". 
   
   Pushed https://github.com/apache/airflow/pull/20878/commits/a83dba737b8685e4e9870af3b0ee1fc5cad82eb3 to add those upper bounds




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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org