kaxil commented on code in PR #35612:
URL: https://github.com/apache/airflow/pull/35612#discussion_r1394905108
##########
setup.cfg:
##########
@@ -163,6 +163,7 @@ install_requires =
# See https://github.com/apache/airflow/pull/31693
# We should also remove "licenses/LICENSE-unicodecsv.txt" file when we
remove this dependency
unicodecsv>=0.14.1
+ universal_pathlib>=0.1.4
Review Comment:
>We could argue we want to pin it specifically to this version to be more
predictable. The downside of that is that upath might pin a particular
dependency (e.g. fsspec) in the future (it doesn't now) to something we do not
like.
That's definitely a big downside
--
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]
