Skip to content

Remove metrics#914

Merged
cleptric merged 2 commits intomasterfrom
remove-metrics
Dec 3, 2024
Merged

Remove metrics#914
cleptric merged 2 commits intomasterfrom
remove-metrics

Conversation

@cleptric
Copy link
Member

@cleptric cleptric commented Dec 3, 2024

Sentry's Metrics Beta has ended and metrics can no longer be ingested. Removing the code from the SDK. More info can be found here.

@cleptric cleptric requested a review from ribice December 3, 2024 16:02
@cleptric cleptric self-assigned this Dec 3, 2024
@codecov
Copy link

codecov bot commented Dec 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.48%. Comparing base (4ee6444) to head (a57c645).
Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #914      +/-   ##
==========================================
+ Coverage   83.16%   84.48%   +1.32%     
==========================================
  Files          55       54       -1     
  Lines        5899     5622     -277     
==========================================
- Hits         4906     4750     -156     
+ Misses        822      704     -118     
+ Partials      171      168       -3     

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

@cleptric cleptric enabled auto-merge (squash) December 3, 2024 16:05
@cleptric cleptric merged commit 6b77510 into master Dec 3, 2024
@cleptric cleptric deleted the remove-metrics branch December 3, 2024 16:09
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