-
-
Notifications
You must be signed in to change notification settings - Fork 23
fix: node filename format was wrong #739
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
Conversation
@tylersayshi is attempting to deploy a commit to the Sentry Team on Vercel. A member of the Team first needs to authorize it. |
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 catch, thanks!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #739 +/- ##
==========================================
- Coverage 31.75% 31.75% -0.01%
==========================================
Files 95 95
Lines 5961 5962 +1
Branches 118 118
==========================================
Hits 1893 1893
- Misses 4068 4069 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or [setup this action to publish automatically](https://github.com/changesets/action#with-publishing). If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @spotlightjs/[email protected] ### Minor Changes - feat: Keep profile spans collapsed ([#737](#737)) ### Patch Changes - fix: Null-check sentryClient.\_options ([#736](#736)) - Updated dependencies \[[`3d569f30fb746d90ecabaac35d23d980360ea99c`](3d569f3), [`d3a2f0a0fae9074802b0551f3e1662833c1423c9`](d3a2f0a), [`d0315bc2ead9ccea722cc73c6b5fd7d9fed3f4a4`](d0315bc)]: - @spotlightjs/[email protected] ## @spotlightjs/[email protected] ### Minor Changes - feat: Keep profile spans collapsed ([#737](#737)) ### Patch Changes - fix: Null-check sentryClient.\_options ([#736](#736)) - Updated dependencies \[[`3d569f30fb746d90ecabaac35d23d980360ea99c`](3d569f3), [`7ea7de17b7ccf9f0c8edb8b413176d2f3482780c`](7ea7de1), [`263ba283efa247e05a022d7f1305cbf819e3e783`](263ba28), [`7ac9fd255cfe85d66e79d632e1d309c3e88d8092`](7ac9fd2), [`a76b2dadb28ce8699c80fc81b709050655bd4629`](a76b2da), [`d3a2f0a0fae9074802b0551f3e1662833c1423c9`](d3a2f0a), [`f2a48b05a41f80e08d1666247f7ccae60bc3d9e7`](f2a48b0), [`6d26f0d21b3ae75e3d81e48ceb2d8f727a94420f`](6d26f0d), [`a8f632357d9dcc46187b00724c14dd4423dfa467`](a8f6323), [`9888dbfc6778de910a2aeae9f3e86f0b2f716a18`](9888dbf), [`ced3e736dfef15d368cac202a10eec4eba7508be`](ced3e73)]: - @spotlightjs/[email protected] - @spotlightjs/[email protected] ## @spotlightjs/[email protected] ### Minor Changes - feat: Keep profile spans collapsed ([#737](#737)) - Swap "Errors" and "Explore" tab places, making "Explore" the default. It makes more sense this way as there is almost ([#758](#758)) guaranteed to have traces in a good setup but not really errors. - Restructured Sentry integration tabs and components ([#765](#765)) - - \#731: Search Bar sticky and fixed overflow across the overlay. ([#740](#740)) - Include filter functionality when searching spans ([#747](#747)) ### Patch Changes - Fixed parsing of envelope data. ([#751](#751)) - fix: Null-check sentryClient.\_options ([#736](#736)) - format node internals correctly in stacktrace ([#739](#739)) - Fixed #738 ([#741](#741)) ## @spotlightjs/[email protected] ### Minor Changes - feat: Keep profile spans collapsed ([#737](#737)) ### Patch Changes - fix: Null-check sentryClient.\_options ([#736](#736)) - Remove obsolete packages from dependencies ([#761](#761)) - Updated dependencies \[[`3d569f30fb746d90ecabaac35d23d980360ea99c`](3d569f3), [`7ea7de17b7ccf9f0c8edb8b413176d2f3482780c`](7ea7de1), [`263ba283efa247e05a022d7f1305cbf819e3e783`](263ba28), [`7ac9fd255cfe85d66e79d632e1d309c3e88d8092`](7ac9fd2), [`a76b2dadb28ce8699c80fc81b709050655bd4629`](a76b2da), [`d3a2f0a0fae9074802b0551f3e1662833c1423c9`](d3a2f0a), [`f2a48b05a41f80e08d1666247f7ccae60bc3d9e7`](f2a48b0), [`6d26f0d21b3ae75e3d81e48ceb2d8f727a94420f`](6d26f0d), [`a8f632357d9dcc46187b00724c14dd4423dfa467`](a8f6323), [`9888dbfc6778de910a2aeae9f3e86f0b2f716a18`](9888dbf), [`ced3e736dfef15d368cac202a10eec4eba7508be`](ced3e73)]: - @spotlightjs/[email protected] - @spotlightjs/[email protected] ## @spotlightjs/[email protected] ### Patch Changes - use nanosecond timestamp for captured filenames ([#776](#776)) - Improve DX by always showing the Spotlight URL, even if the sidecar was already running. Makes it easy to cmd/ctrl+click ([#748](#748)) and open in browser. Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Very minor quirk noticed with the stacktrace... node internals were being formatted incorrectly by
formatFilename
, so this change just returns them as is, since they are formatted well to begin with.before:
after: