dvc icon indicating copy to clipboard operation
dvc copied to clipboard

add dvc artifacts get --show-url

Open dberenbaum opened this issue 1 year ago • 2 comments

Closes #10420

dberenbaum avatar May 10 '24 14:05 dberenbaum

Codecov Report

Attention: Patch coverage is 20.00000% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 90.60%. Comparing base (a7c0f6e) to head (a23367d). Report is 1 commits behind head on main.

Files Patch % Lines
dvc/commands/artifacts.py 20.00% 8 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10423      +/-   ##
==========================================
+ Coverage   90.50%   90.60%   +0.10%     
==========================================
  Files         502      502              
  Lines       38982    38992      +10     
  Branches     5622     5632      +10     
==========================================
+ Hits        35279    35329      +50     
+ Misses       3006     2991      -15     
+ Partials      697      672      -25     

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

codecov[bot] avatar May 10 '24 14:05 codecov[bot]

This looks good... @dberenbaum does it need the remote git repo access or can it rely on the token? I think that either is fine for the moment but we should clarify that in the docs (since if it needs the repo access it behaves differently than other uses of dvc artifacts get

tibor-mach avatar May 13 '24 10:05 tibor-mach

Can this be merged now?

tibor-mach avatar May 23 '24 16:05 tibor-mach