Skip to content

ref!: remove Experimental from Metrics API#5023

Merged
jamescrosswell merged 2 commits intomainfrom
ref/metrics-stable-api
Mar 16, 2026
Merged

ref!: remove Experimental from Metrics API#5023
jamescrosswell merged 2 commits intomainfrom
ref/metrics-stable-api

Conversation

@Flash0ver
Copy link
Member

@Flash0ver Flash0ver commented Mar 13, 2026

Summary

Remove "Experimental" properties and [ExperimentalAttribute] from all Metrics APIs.

Remarks

Changelog Entry

The Metrics APIs are now stable: removed Experimental from SentrySdk, SentryOptions and IHub

Issues

Closes #4921

@Flash0ver Flash0ver self-assigned this Mar 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Breaking Changes 🛠

  • The _Metrics_ APIs are now stable: removed Experimental from SentrySdk, SentryOptions and IHub by Flash0ver in #5023

Dependencies ⬆️

Deps

  • chore(deps): update Java SDK to v8.35.0 by github-actions in #5017
  • chore(deps): replaced the heavy protobuf-javalite 3.25.8 dependency with a light-weight epitaph 0.1.0 alternative on Android (getsentry/sentry-java#5157) by github-actions in #5017
  • chore(deps): update CLI to v3.3.3 by github-actions in #5002
  • chore(deps): update Cocoa SDK to v9.6.0 by github-actions in #4958

🤖 This preview updates automatically when you update the PR.

@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.95%. Comparing base (bfe5319) to head (369e8df).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5023      +/-   ##
==========================================
+ Coverage   73.92%   73.95%   +0.02%     
==========================================
  Files         497      497              
  Lines       17974    17966       -8     
  Branches     3517     3516       -1     
==========================================
- Hits        13287    13286       -1     
+ Misses       3832     3823       -9     
- Partials      855      857       +2     

☔ 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 marked this pull request as ready for review March 13, 2026 15:53
@jamescrosswell
Copy link
Collaborator

This again:

/Users/runner/work/sentry-dotnet/sentry-dotnet/src/Sentry.Bindings.Cocoa/Sentry.Bindings.Cocoa.csproj(112,11): error MSB4006: There is a circular dependency in the target dependency graph involving target "SetupCocoaSDK". [TargetFramework=net9.0-ios18.0]
/Users/runner/work/sentry-dotnet/sentry-dotnet/src/Sentry.Bindings.Cocoa/Sentry.Bindings.Cocoa.csproj(112,11): error MSB4006: There is a circular dependency in the target dependency graph involving target "SetupCocoaSDK". [TargetFramework=net9.0-maccatalyst18.0]

I bump into this once or twice a day... just costs time requeusting gh rerun the jobs and then coming back later to do actual reviews/approvals.

It should hopefully be addressed by #4956. @Flash0ver I think we should get that merged in to check (we can always revert if it doesn't work).

@jamescrosswell jamescrosswell merged commit 275bcef into main Mar 16, 2026
58 of 59 checks passed
@jamescrosswell jamescrosswell deleted the ref/metrics-stable-api branch March 16, 2026 00:54
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.

feat!: remove Experimental from Metrics APIs

2 participants