Skip to content

ci(craft): temporary remove new NuGet package for initial release#5192

Merged
Flash0ver merged 1 commit intomainfrom
ci/temporary-remove-new-nuget-package-for-release
May 5, 2026
Merged

ci(craft): temporary remove new NuGet package for initial release#5192
Flash0ver merged 1 commit intomainfrom
ci/temporary-remove-new-nuget-package-for-release

Conversation

@Flash0ver
Copy link
Copy Markdown
Member

@Flash0ver Flash0ver commented May 5, 2026

Summary

Temporary remove nuget:Sentry.OpenTelemetry.Exporter from Craft.

Remarks

I ran into a chicken-or-the-egg problem when trying to publish release 6.5.0 which includes a new SDK / NuGet package:

  1. feat: Add support to send OTEL traces via OTLP #4899
    • added to package .craft.yml
    • nuget:Sentry.OpenTelemetry.Exporter:
  2. publish: getsentry/sentry-dotnet@6.5.0 publish#8066
  3. feat(dotnet): add NuGet package Sentry.OpenTelemetry.Exporter sentry-release-registry#234
    • manually added Sentry.OpenTelemetry.Exporter to sentry-release-registry
  4. publish: getsentry/sentry-dotnet@6.5.0 publish#8066

To fix the issue:

  1. remove nuget:Sentry.OpenTelemetry.Exporter: from .craft.yml
    • this PR
  2. publish the release again
  3. add nuget:Sentry.OpenTelemetry.Exporter: to .craft.yml

Instead - for the next time we publish a new package - before we publish the new package, we should do:


#skip-changelog

@Flash0ver Flash0ver self-assigned this May 5, 2026
@Flash0ver Flash0ver marked this pull request as ready for review May 5, 2026 13:22
@Flash0ver Flash0ver requested a review from jamescrosswell as a code owner May 5, 2026 13:22
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.08%. Comparing base (14c1d8c) to head (249234a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5192   +/-   ##
=======================================
  Coverage   74.08%   74.08%           
=======================================
  Files         506      506           
  Lines       18247    18247           
  Branches     3564     3564           
=======================================
+ Hits        13518    13519    +1     
  Misses       3858     3858           
+ Partials      871      870    -1     

☔ 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.

@Flash0ver Flash0ver added the skip-changelog Suppress automatic changelog generation via Craft label May 5, 2026
@Flash0ver Flash0ver merged commit 3f60554 into main May 5, 2026
40 checks passed
@Flash0ver Flash0ver deleted the ci/temporary-remove-new-nuget-package-for-release branch May 5, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Suppress automatic changelog generation via Craft

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants