potiuk commented on a change in pull request #20910:
URL: https://github.com/apache/airflow/pull/20910#discussion_r786312723



##########
File path: setup.cfg
##########
@@ -86,9 +85,8 @@ install_requires =
     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, <7.0
+    connexion[swagger-ui,flask]>=2.10.0

Review comment:
       Good question - that's why I need some coments from @mik-laj  and 
@ephraimbuddy @uranusjr. 
   
   I just added it back from the original connexion vendoring commit: 
https://github.com/apache/airflow/commit/bb115da0b78036ace5e31e2139c12ca044b3a97d#diff-fa602a8a75dc9dcc92261bac5f533c2a85e34fcceaff63b3a3a81d9acde2fc52L90
 
   
   My guess it's needed to simply integrate connexion routes with flask. 




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


Reply via email to