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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
