Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Nov 20, 2024

We had this odd sampling frequency value in our docs which looks like a typo but it is not. This PR adds a note box to reassure the readers with some explanation and a reference link to why.

We had this odd sampling frequency value in our docs which looks like a typo but it is not. This PR adds a note box to reassure the readers with some explanation and a reference link to why.
@vercel
Copy link

vercel bot commented Nov 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
develop-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 8:33pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Nov 20, 2024 8:33pm
sentry-docs ⬜️ Ignored (Inspect) Visit Preview Nov 20, 2024 8:33pm

@BYK BYK requested a review from viglia November 20, 2024 12:52
@BYK BYK enabled auto-merge (squash) November 20, 2024 14:21
@armcknight armcknight changed the title feat(profilin): Add explanation for lockstep sampling feat(profiling): Add explanation for lockstep sampling Nov 20, 2024
Co-authored-by: Andrew McKnight <amcknight2718@gmail.com>
@BYK BYK merged commit 10cd1e7 into master Nov 21, 2024
10 checks passed
@BYK BYK deleted the byk/feat/add-sample-rate-context branch November 21, 2024 10:21
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants