Skip to content

Conversation

@skshetry
Copy link
Collaborator

Also replaces incorrect usages of as_tqdm_callback. as_tqdm_callback are meant to be used when you want to use a callback that is passed or default to TqdmCallback. Which does not seem to apply here.

@codecov
Copy link

codecov bot commented Dec 13, 2023

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (3c31234) 90.63% compared to head (51a3f56) 90.33%.
Report is 1 commits behind head on main.

Files Patch % Lines
dvc/repo/artifacts.py 0.00% 3 Missing ⚠️
dvc/repo/worktree.py 0.00% 3 Missing ⚠️
dvc/repo/imports.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10158      +/-   ##
==========================================
- Coverage   90.63%   90.33%   -0.30%     
==========================================
  Files         496      500       +4     
  Lines       37754    37850      +96     
  Branches     5492     5503      +11     
==========================================
- Hits        34218    34192      -26     
- Misses       2900     2990      +90     
- Partials      636      668      +32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@skshetry skshetry merged commit dec1ac7 into treeverse:main Dec 13, 2023
@skshetry skshetry deleted the dvc-data-update branch December 13, 2023 11:10
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.

1 participant