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
feat(onboarding): Add hapi, fastify and Nest.js #70647
Conversation
0a0569f
to
dc2ae99
Compare
Updates the code snippets to show the new serverless packages for aws and gcp.
For JS SDK v8 an external file is needed to initialize the SDK right at the start of the application. The existing node guides are updated. --------- Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
Improve tests for onboarding pages to make sure the profiling import is present and the correct error handler is used.
65056c4
to
fead1c8
Compare
|
||
import * as Sentry from '@sentry/node'; |
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.
import * as Sentry from '@sentry/node'; | |
// All other imports below | |
import * as Sentry from '@sentry/node'; |
let's add a comment like this, this ensures this is not reordered e.g. by vscode!
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.
Great work, looks perfect!
Bundle ReportChanges will increase total bundle size by 104.6kB ⬆️
|
Will merge this after v8 is out for a while (like 1 week or so). --------- Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Will merge this after v8 is out for a while (like 1 week or so).