potiuk commented on code in PR #38531:
URL: https://github.com/apache/airflow/pull/38531#discussion_r1570592867


##########
hatch_build.py:
##########
@@ -414,6 +414,7 @@
     # Blinker use for signals in Flask, this is an optional dependency in 
Flask 2.2 and lower.
     # In Flask 2.3 it becomes a mandatory dependency, and flask signals are 
always available.
     "blinker>=1.6.2",
+    "cloudpickle",

Review Comment:
   We should not make cloudpickle required dependency - we know dill is needed 
for executor config, but we should rather leave cloudpickle as optional 
dependency only.



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