Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move fs utilities to fs.py from __init__.py #3093

Merged
merged 3 commits into from
Jan 9, 2020
Merged

Conversation

algomaster99
Copy link
Contributor

  • ❗ Have you followed the guidelines in the Contributing to DVC list?

  • πŸ“– Check this box if this PR does not require documentation updates, or if it does and you have created a separate PR in dvc.org with such updates (or at least opened an issue about it in that repo). Please link below to your PR (or issue) in the dvc.org repo.

  • ❌ Have you checked DeepSource, CodeClimate, and other sanity checks below? We consider their findings recommendatory and don't expect everything to be addressed. Please review them carefully and fix those that actually improve code or fix bugs.

Thank you for the contribution - we'll try to review it as soon as possible. πŸ™

This pull request moves all fs utilities to fs.py from __init__.py.

Fixes #2137

@algomaster99 algomaster99 requested a review from efiop January 8, 2020 23:31
@algomaster99
Copy link
Contributor Author

@efiop I have initiated the shifting of filesystem utilities from __init__.py to fs.py. Tell me if there are more.

@codecov
Copy link

codecov bot commented Jan 8, 2020

Codecov Report

Merging #3093 into master will increase coverage by 1.69%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3093      +/-   ##
==========================================
+ Coverage   90.07%   91.77%   +1.69%     
==========================================
  Files         138      138              
  Lines        8535     8535              
==========================================
+ Hits         7688     7833     +145     
+ Misses        847      702     -145
Impacted Files Coverage Ξ”
dvc/utils/__init__.py 93.37% <ΓΈ> (-1.02%) ⬇️
dvc/utils/fs.py 96.85% <100%> (+1.01%) ⬆️
dvc/remote/base.py 95% <100%> (+0.19%) ⬆️
dvc/remote/local.py 92.98% <100%> (-0.06%) ⬇️
dvc/analytics.py 95.89% <100%> (+2.83%) ⬆️
dvc/repo/__init__.py 97.48% <100%> (ΓΈ) ⬆️
dvc/updater.py 54.65% <0%> (-1.17%) ⬇️
dvc/remote/ssh/connection.py 84.15% <0%> (+0.49%) ⬆️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update 277d28c...74850fe. Read the comment docs.

@efiop
Copy link
Contributor

efiop commented Jan 9, 2020

@algomaster99 Thank you for the PR! πŸ™ I think the only one that is left to move is walk_files and that should be it πŸ™‚

@algomaster99 algomaster99 changed the title WIP: Move fs utilities to fs.py from __init__.py Move fs utilities to fs.py from __init__.py Jan 9, 2020
@Suor Suor mentioned this pull request Jan 9, 2020
2 tasks
@algomaster99
Copy link
Contributor Author

@efiop PR has been updated. Please see. Thanks!

@efiop
Copy link
Contributor

efiop commented Jan 9, 2020

Thank you @algomaster99 ! πŸ™

For the record: tests failing for unrelated reasons.

@efiop efiop merged commit 095464d into master Jan 9, 2020
@delete-merged-branch delete-merged-branch bot deleted the move-fs-utilities branch January 9, 2020 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dvc: update file utilities
2 participants