[OpAMP.Client] Remove System.Net.Http NuGet reference by RassK · Pull Request #4402 · open-telemetry/opentelemetry-dotnet-contrib (original) (raw)

Fixes issues with net462 System.Net.Http version resolving.

Changes

• Uses framework reference instead of NuGet package.

Merge requirement checklist

• CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
• Appropriate CHANGELOG.md files updated for non-trivial changes

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.15%. Comparing base (e05a4ba) to head (d6c4523).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@ Coverage Diff @@ ## main #4402 +/- ##

• Coverage 76.22% 76.15% -0.08%

  Files 455 455
  Lines 18334 18334

• Hits 13975 13962 -13
• Misses 4359 4372 +13

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})