Skip to content

Conversation

@bitsandfoxes
Copy link
Contributor

@vercel
Copy link

vercel bot commented Feb 11, 2026

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

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Feb 11, 2026 2:41pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Feb 11, 2026 2:41pm

Request Review

@codeowner-assignment codeowner-assignment bot requested a review from a team February 11, 2026 14:34
@@ -0,0 +1,42 @@
Metrics are enabled by default. Once you initialize the SDK, you can send metrics using the `SentrySdk.Experimental.Metrics` APIs.
Copy link

Choose a reason for hiding this comment

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

Bug: The Unity metrics usage documentation incorrectly states that metrics are enabled by default, but the setup guide requires explicit enablement, leading to silent failures for users.
Severity: MEDIUM

Suggested Fix

In platform-includes/metrics/usage/unity.mdx, change the first line from "Metrics are enabled by default. Once you initialize the SDK..." to "Once metrics are enabled, you can send metrics...". This removes the false claim and directs users to the setup guide implicitly.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: platform-includes/metrics/usage/unity.mdx#L1

Potential issue: The documentation file `platform-includes/metrics/usage/unity.mdx`
claims "Metrics are enabled by default." However, the corresponding setup documentation
(`platform-includes/metrics/setup/unity.mdx`) provides explicit steps to enable them,
implying they are disabled by default. This contradiction will cause developers who
follow the usage guide to see their metrics fail to be sent silently, as they will miss
the required setup step. This leads to a confusing user experience and wasted debugging
time.

Did we get this right? 👍 / 👎 to inform future reviews.

@bitsandfoxes bitsandfoxes merged commit bc0b6bd into master Feb 11, 2026
14 checks passed
@bitsandfoxes bitsandfoxes deleted the feat/unity-metrics branch February 11, 2026 15:52
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