bump: bump `oras-go` to 2.5.0

oras-project / oras

OCI registry client - managing content like artifacts, images, packages

https://oras.land

Apache License 2.0

1.33k stars 161 forks source link

bump: bump `oras-go` to 2.5.0 #1303

Closed wangxiaoxuan273 closed 3 months ago

wangxiaoxuan273 commented 3 months ago

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Part of #1275 Fixes #1304

Please check the following list:

[x] Does the affected code have corresponding tests, e.g. unit test, E2E test?
[ ] Does this change require a documentation update?
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have an appropriate license header?

qweeah commented 3 months ago

@shizhMSFT is right, you need to 1) handle all invalid reference error in tag list reflecting https://github.com/oras-project/oras/pull/1058; 2) improve target.Parse to accommodate changes in oras-go.

qweeah commented 3 months ago

Also need to refactor reference parsing for OCI image layout and do it in target.Parse, so unable to add tag for '%s' can be printed for OCI layout

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.27%. Comparing base (24c2acf) to head (39155e8).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #1303 +/- ## ========================================== + Coverage 82.10% 82.27% +0.16% ========================================== Files 83 83 Lines 4030 4029 -1 ========================================== + Hits 3309 3315 +6 + Misses 498 494 -4 + Partials 223 220 -3 ```

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

wangxiaoxuan273 commented 3 months ago

Since this PR also fixes #1304, we also need to enhance E2E tests.

Added a test case for #1304