search

oras-project / oras

OCI registry client - managing content like artifacts, images, packages
https://oras.land
Apache License 2.0
1.52k stars 181 forks source link

refactor: Create text copy handler #1441

Closed TerryHowe closed 4 months ago

TerryHowe commented 5 months ago

What this PR does / why we need it:

This is a baby step to creating copy handlers for the cp command

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.

Project coverage is 85.48%. Comparing base (7a88524) to head (361025d).

Files Patch % Lines
cmd/oras/internal/display/status/text.go 88.88% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1441 +/- ## ========================================== - Coverage 85.52% 85.48% -0.05% ========================================== Files 109 109 Lines 3842 3851 +9 ========================================== + Hits 3286 3292 +6 - Misses 333 335 +2 - Partials 223 224 +1 ```

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

TerryHowe commented 4 months ago

Still need to add TTY implementation of status.CopyHandler. Not necessarily needed in this PR.

If this PR merges, it will be easier to create a tty handler and the successor PR is so bloated.