xorbitsai / xorbits

Scalable Python DS & ML, in an API compatible & lightning fast way.
https://xorbits.readthedocs.io
Apache License 2.0
1.11k stars 67 forks source link

BUG: fix s3 glob #612

Closed Hank0626 closed 1 year ago

Hank0626 commented 1 year ago

What do these changes do?

Related issue number

Fixes #607

Check code requirements

codecov[bot] commented 1 year ago

Codecov Report

Merging #612 (86f5a4e) into main (7690e78) will increase coverage by 0.01%. The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #612      +/-   ##
==========================================
+ Coverage   93.54%   93.56%   +0.01%     
==========================================
  Files        1012     1012              
  Lines       78738    78737       -1     
  Branches    16361    16358       -3     
==========================================
+ Hits        73659    73672      +13     
+ Misses       3419     3401      -18     
- Partials     1660     1664       +4     
Flag Coverage Δ
unittests 93.46% <66.66%> (+0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
python/xorbits/_mars/lib/filesystem/_glob.py 88.46% <66.66%> (-4.00%) :arrow_down:

... and 5 files with indirect coverage changes