docs(cloudflare): Remove install for Nuxt/SvelteKit#14315
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 375 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
|
inventarSarah
left a comment
There was a problem hiding this comment.
LGTM!
I've added two suggestions to change some wording -- I couldn't insert the comments directly into the relevant lines; please let me know if it's unclear what I'm suggesting there :)
|
|
||
| </Alert> | ||
|
|
||
| Now you can install the Sentry Cloudflare SDK with your package manager: |
There was a problem hiding this comment.
Now that this is gone, we don't need to write "First, install the Sentry ..." in the first sentence under the "Install" heading.
I think we can just combine the first two sentences:
## Install
We recommend using the Sentry wizard to automatically install the SDK in your application:
|
|
||
| </Alert> | ||
|
|
||
| Now you can install the Sentry Cloudflare SDK. First, install the SDK with your package manager: |
There was a problem hiding this comment.
Same as for Nuxt:
Now that this is gone, we don't need to write "First, install the Sentry ..." in the first sentence under the "Install" heading.
I think we can just combine the first two sentences:
## Install
We recommend using the Sentry wizard to automatically install the SDK in your application:
DESCRIBE YOUR PR
Remove the Cloudflare install as it's automatically installed.
Merged after this is released: getsentry/sentry-javascript#16873
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: