Skip to content

docs(metrics): remove unsupported set type#16056

Merged
Flash0ver merged 3 commits intomasterfrom
docs/remove-set-from-metrics
Jan 19, 2026
Merged

docs(metrics): remove unsupported set type#16056
Flash0ver merged 3 commits intomasterfrom
docs/remove-set-from-metrics

Conversation

@Flash0ver
Copy link
Member

@Flash0ver Flash0ver commented Jan 16, 2026 •
edited
Loading

DESCRIBE YOUR PR

During getsentry/sentry-dotnet#4834 and #16054, I noticed that the Metrics dev docs, and the JavaScript user docs mention the Metric-Type set. This type is currently not supported.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

@vercel
Copy link

vercel bot commented Jan 16, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
develop-docs Ready Ready Preview, Comment Jan 19, 2026 10:40am
sentry-docs Ready Ready Preview, Comment Jan 19, 2026 10:40am

Request Review

@codeowner-assignment codeowner-assignment bot requested a review from a team January 16, 2026 19:30
@Flash0ver Flash0ver marked this pull request as ready for review January 16, 2026 20:06
@Flash0ver Flash0ver self-assigned this Jan 16, 2026
antonis
antonis approved these changes Jan 19, 2026
View reviewed changes
Copy link
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for this change 🙇

@codeowner-assignment codeowner-assignment bot requested a review from a team January 19, 2026 10:29
@Flash0ver Flash0ver merged commit c31f3af into master Jan 19, 2026
14 checks passed
@Flash0ver Flash0ver deleted the docs/remove-set-from-metrics branch January 19, 2026 10:53
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@antonis antonis antonis approved these changes

@chargome chargome chargome approved these changes

@nicohrubec nicohrubec nicohrubec approved these changes

@alexander-alderman-webb alexander-alderman-webb alexander-alderman-webb approved these changes

Assignees

@Flash0ver Flash0ver

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Flash0ver @antonis @chargome @nicohrubec @alexander-alderman-webb