Skip to content

Conversation

@martincostello
Copy link
Owner

Consume changes from open-telemetry/opentelemetry-dotnet#6567 for testing.

Copilot AI review requested due to automatic review settings October 6, 2025 13:34
@martincostello martincostello added dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code labels Oct 6, 2025
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 updates OpenTelemetry packages to consume changes from an upstream pull request in the opentelemetry-dotnet repository for testing purposes. The changes enable local package testing by configuring NuGet to use a local package source and updating OpenTelemetry packages to pre-release alpha versions.

  • Adds local package source configuration to NuGet.config
  • Updates OpenTelemetry packages to alpha versions (1.13.1-alpha.0.11)

Reviewed Changes

Copilot reviewed 2 out of 12 changed files in this pull request and generated no comments.

File Description
NuGet.config Adds local package source and mapping for OpenTelemetry packages
Directory.Packages.props Updates OpenTelemetry package versions to pre-release alpha builds

@martincostello martincostello enabled auto-merge (rebase) October 6, 2025 13:36
@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.68%. Comparing base (22ad75b) to head (7b1ec4c).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2797      +/-   ##
==========================================
+ Coverage   86.34%   86.68%   +0.34%     
==========================================
  Files         115      115              
  Lines        4349     4349              
  Branches      500      500              
==========================================
+ Hits         3755     3770      +15     
+ Misses        464      449      -15     
  Partials      130      130              
Flag Coverage Δ
linux 86.38% <ø> (+0.27%) ⬆️
macos 86.27% <ø> (ø)
windows 86.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Fix package restore for packages that prereleases are not being tested for.
@martincostello martincostello enabled auto-merge (squash) October 6, 2025 13:41
Copy link
Collaborator

@costellobot costellobot left a comment

Choose a reason for hiding this comment

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

Device URL
desktop https://localhost:50001/sign-in

Device URL
mobile https://localhost:50001/sign-in

Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead

@martincostello martincostello merged commit dbbab2c into main Oct 6, 2025
18 checks passed
@martincostello martincostello deleted the otel-6567 branch October 6, 2025 13:46
martincostello added a commit that referenced this pull request Oct 6, 2025
martincostello added a commit that referenced this pull request Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants