-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New test analytics doc for docs.sentry #14296
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Bundle ReportChanges will decrease total bundle size by 9.64kB (-0.04%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
|
I went ahead and set this to draft since it looks like there's still some design assets to be added. |
3ecd76b to
fe3b16d
Compare
7b950d0 to
52d625b
Compare
|
|
||
| - **Spot Flaky Tests:** Quickly identify and address tests that fail intermittently, reducing noise and wasted debugging time | ||
| - **Track Failures and Trends:** Monitor cumulative failures, slowest tests, and overall test run efficiency to improve CI reliability | ||
| - **Actionable Insights:** See which lines of code failed within tests, confirm flaky tests, and prioritize fixes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is 'confirm flaky tests' different from 'spot flaky tests' above?
| sha256sum <filename> | ||
| ``` | ||
|
|
||
| and compare the checksum generated with that from GitHub releases. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to have a screenshot or code snippet of what a successful execution would look like here, so the user knows if they got the right result?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that might be possible. I haven't had a chance to test it yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Added some suggestions, the only blocker is the broken relative link.
Co-authored-by: Alex Krawiec <[email protected]>
Co-authored-by: Alex Krawiec <[email protected]>
Co-authored-by: Alex Krawiec <[email protected]>
Co-authored-by: Alex Krawiec <[email protected]>
Co-authored-by: Alex Krawiec <[email protected]>
|
@coolguyzone thanks for catching that link! It turns out both external were kind of broken, so I was able to fix them properly. I'll create a follow up for the integrity check. |
DESCRIBE YOUR PR
Sentry Prevent is adding in Test Analytics to our product.
sentry-prevent-ai: https://sentry-docs-git-sentry-prevent-test-analytics-docs.sentry.dev/product/ai-in-sentry/ai-code-review/ (should redirect when on https://sentry-docs-git-sentry-prevent-test-analytics-docs.sentry.dev/product/ai-in-sentry/sentry-prevent-ai/ )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: