-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Adding sentry prevent AI docs #13884
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
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Bundle ReportChanges will increase total bundle size by 150 bytes (0.0%) ⬆️. 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:
|
Sentry Prevent AI is a generative AI assistant developed by Sentry. It helps you generate new tests for uncovered code and reviews your code changes, offering suggestions for improvement before merging pull requests. | ||
|
||
<Alert> | ||
Sentry Prevent AI is currently in <strong>beta</strong>. Beta features are still a work in progress and may have bugs. Please reach out on GitHub Issues if you have any feedback or concerns. |
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.
Is it worth adding a link to the place where you can submit a github issue?
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 so - if we're still taking feedback!
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.
Could the headline change? suggestion:
Get code reviews and tests for code that breaks less using Sentry Prevent AI
Also it mentions the GitHub app but no link is provided
@lindsayrpiper - yeah, happy to update the headline. I'm waiting on what the proper link is. @rohan-at-sentry - is that github app link this one? https://github.com/apps/sentry-io |
@sfanahata thanks for reminding 😬 https://github.com/apps/seer-by-sentry is the app link Also there are some changes to the triggers. Will mark inline and request changes |
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.
we're pushing the updates to the prompts - The comment should describe what's changing.
|
||
After installing the app, use these commands in your pull request comments: | ||
|
||
- `@sentry-prevent-ai-reviewer review` — The assistant will review the PR and make suggestions. |
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.
@sentry review
- The assistant will review the PR and make suggestions
After installing the app, use these commands in your pull request comments: | ||
|
||
- `@sentry-prevent-ai-reviewer review` — The assistant will review the PR and make suggestions. | ||
- `@sentry-prevent-ai-reviewer test` — The assistant will generate tests for your PR. |
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.
@sentry generate-test
- The assistant will generate tests for your PR
DESCRIBE YOUR PR
For the Sentry Prevent AI beta releasing in June
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: