-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
docs(js): Update Bun quick start guide #15419
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 GitHub.
1 Skipped Deployment
|
Bundle ReportChanges will decrease total bundle size by 3.22kB (-0.01%) ⬇️. 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:
|
39aaf7b to
e7ff644
Compare
e7ff644 to
02adf8c
Compare
02adf8c to
d052484
Compare
| Now's a good time to customize your setup and look into more advanced topics. | ||
| Our next recommended steps for you are: | ||
|
|
||
| - Extend Sentry to your frontend using one of our [frontend SDKs](/) |
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.
just double-checking, this is meant to go to the docs homepage?
coolguyzone
left a comment
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.
Nice work!
| Import `instrument.js` before any other modules to ensure Sentry initializes early. If you initialize later, auto-instrumentation and modules like database monitoring, agent monitoring, tracing may not work. | ||
|
|
||
| </Alert> | ||
| Import `instrument.js` before any other modules to make sure Sentry initializes early. If you initialize later, auto-instrumentation and modules like tracing may not work. |
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.
Bun has the same problem as Node with ESM (in Node, we have the "Installation methods"). In bun we would need to --preload this instrumentation file to be loaded before everything else: https://bun.com/docs/runtime#param-preload
This is out of scope for this PR but I just want to mention that. We need to improve our docs to be less Node-focused and also include other runtimes such as Bun and Deno in the future.
DESCRIBE YOUR PR
This branch contains the updated Bun Quick Start guide, following our template.
Closes: #15345
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:
EXTRA RESOURCES