search

aai-institute / lakefs-spec

An fsspec implementation for the lakeFS project.
http://lakefs-spec.org/
Apache License 2.0
37 stars 4 forks source link

Bump minimum fsspec version to 2023.12.0 #280

Closed nicholasjng closed 1 month ago

nicholasjng commented 1 month ago

This means that the polymorphic filesystem facility, which was upstreamed to fsspec and first appeared in that release, can go away.

Contains the error handling fix from #278 as a cherrypick.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.54%. Comparing base (4c4a32f) to head (2491840).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #280 +/- ## ========================================== + Coverage 94.37% 94.54% +0.17% ========================================== Files 5 5 Lines 391 385 -6 Branches 73 72 -1 ========================================== - Hits 369 364 -5 Misses 13 13 + Partials 9 8 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

AdrianoKF commented 1 month ago

We'll have to see about the changes to coverage in the error handling code, but I'd leave that up to the other PR #278.