docs(godot): Document Metrics general availability in SDK 2.0.0#18496
Open
limbonaut wants to merge 4 commits into
Open
docs(godot): Document Metrics general availability in SDK 2.0.0#18496limbonaut wants to merge 4 commits into
limbonaut wants to merge 4 commits into
Conversation
Metrics are generally available and supported on all platforms (including Apple) as of Godot SDK 2.0.0. The enable_metrics and before_send_metric options moved out of the experimental namespace onto SentryOptions. - Remove the note that metrics aren't supported on Apple platforms - Update the Project Settings path from Sentry > Experimental to Sentry > Options - Drop the experimental labels and experimental.* option paths - Bump the requirements note to 2.0.0 (previously experimental since 1.4.0) - Document enable_metrics and before_send_metric on the general options reference Refs getsentry/sentry-godot#755 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Move before_send_metric next to enable_metrics under Metrics Options, and before_send_log next to enable_logs under Logging Options, instead of listing them under the general Hooks section. This groups each feature's toggle and its before-send hook together, matching how the Apple SDK documents metric and log hooks, and keeps the two data hooks consistent with each other. The Hooks section now covers only the event-level hooks (before_send, before_capture_screenshot). Refs getsentry/sentry-godot#755 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
limbonaut
commented
Jun 22, 2026
| <SdkOption name="before_send_log" type="Callable" availableSince="1.2.0"> | ||
|
|
||
| If assigned, this callback will be called before sending a log message to Sentry. | ||
| It can be used to modify the log message or prevent it from being sent. |
Collaborator
Author
There was a problem hiding this comment.
FYI: Moved it under Logging section to align with metrics.
limbonaut
force-pushed
the
limbonaut/docs/godot-metrics-ga
branch
from
53fa14d to
85471ef
Compare
JoshuaMoelans
approved these changes
Jun 22, 2026
JoshuaMoelans
left a comment
JoshuaMoelans
left a comment
Member
There was a problem hiding this comment.
LGTM, just a minor suggestion
Co-authored-by: JoshuaMoelans <60878493+JoshuaMoelans@users.noreply.github.com>
mujacica
approved these changes
Jun 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DESCRIBE YOUR PR
Documents Sentry Metrics for the Godot SDK becoming generally available in SDK
2.0.0, where they are now supported on all platforms (including Apple) and theenable_metrics/before_send_metricoptions move out of the experimental namespace ontoSentryOptions.Warning
This documents the unreleased Godot SDK
2.0.0. Don't merge!IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes: