-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
Bump: @sentry/javascript dependencies to 6.19.2 #2175
Conversation
// release: '[email protected]+1', | ||
// dist: `1`, |
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.
@jennmueng do you remember why did we hard-coded the release
and dist
in the samples? that makes our sample symbolication not work automatically.
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.
@marandaneto That was just to provide an example of the option being used I think
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.
👍
📜 Description
Bump: @sentry/javascript dependencies to 6.19.2
For PR review, focus on 7655547
The rest is lint fixes.
💡 Motivation and Context
Bug fixes getsentry/sentry-javascript#4531 and getsentry/sentry-javascript#4614
💚 How did you test it?
📝 Checklist
🔮 Next steps