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/4bdd6cf1718db88ff02030b8a490b4436aee85f7
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]
