Skip to content

Conversation

@Wwwsylvia
Copy link
Member

What this PR does / why we need it

Dependencies in the 9.* series support .NET 8 and .NET 9, while 8.* supports .NET 6–8. As a library, sticking with 8.* ensures broader compatibility.

Which issue(s) this PR resolves / fixes

Please check the following list

  • 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?

Copilot AI review requested due to automatic review settings October 10, 2025 09:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR downgrades package dependencies from the 9.* series to 8.* versions to improve compatibility. The goal is to support a broader range of .NET versions (.NET 6-8) rather than limiting support to .NET 8-9 only, which is appropriate for a library that aims for maximum compatibility.

Key changes:

  • Downgraded Microsoft.Extensions.Caching.Abstractions from 9.0.9 to 8.0.0
  • Downgraded Microsoft.Extensions.Caching.Memory from 9.0.9 to 8.0.1
  • Downgraded System.Text.Json from 9.0.9 to 8.0.6

@codecov
Copy link

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.84%. Comparing base (5cbb22c) to head (e41bf4b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #279   +/-   ##
=======================================
  Coverage   89.84%   89.84%           
=======================================
  Files          55       55           
  Lines        2392     2392           
  Branches      321      321           
=======================================
  Hits         2149     2149           
  Misses        161      161           
  Partials       82       82           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Lixia (Sylvia) Lei <[email protected]>
Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Wwwsylvia Wwwsylvia merged commit 1804122 into oras-project:main Oct 10, 2025
8 checks passed
@Wwwsylvia Wwwsylvia deleted the deps_version branch October 10, 2025 10:34
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.

2 participants