Skip to content

Comments

Add more Sentry config#465

Merged
trevor-e merged 2 commits intomainfrom
telkins/more-sentry-config
May 16, 2025
Merged

Add more Sentry config#465
trevor-e merged 2 commits intomainfrom
telkins/more-sentry-config

Conversation

@trevor-e
Copy link
Collaborator

No description provided.

@emerge-tools
Copy link

emerge-tools bot commented May 16, 2025

📸 Snapshot Test

1 modified, 71 unchanged

Name Added Removed Modified Renamed Unchanged Errored Approval
HackerNews
com.emergetools.hackernews.snapshots
0 0 1 0 71 0 ✅ Approved

🛸 Powered by Emerge Tools

Copy link

@armcknight armcknight left a comment

Choose a reason for hiding this comment

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

Noice 👍🏻

options.environment = "development"
options.sessionReplay.sessionSampleRate = 1.0
options.tracesSampleRate = 1
// options.debug = true

Choose a reason for hiding this comment

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

Suggested change
// options.debug = true
options.debug = true

Not sure if you meant to comment this for debug builds

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yea I left this commented out for now since it was way noisier than I thought it would be.

@trevor-e trevor-e merged commit cbff701 into main May 16, 2025
6 checks passed
@trevor-e trevor-e deleted the telkins/more-sentry-config branch May 16, 2025 18:25
NicoHinderling pushed a commit that referenced this pull request May 28, 2025
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