-
-
Notifications
You must be signed in to change notification settings - Fork 32.6k
lib: add type names in source mapped stack traces #58976
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
lib: add type names in source mapped stack traces #58976
Conversation
5ed51ea to
a0f517b
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #58976 +/- ##
==========================================
+ Coverage 89.97% 89.99% +0.02%
==========================================
Files 649 649
Lines 192194 192189 -5
Branches 37678 37678
==========================================
+ Hits 172918 172956 +38
+ Misses 11873 11825 -48
- Partials 7403 7408 +5
🚀 New features to boost your workflow:
|
This comment was marked as outdated.
This comment was marked as outdated.
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.
LGTM % a nit: aren't the tests going through source maps by using console.trace instead of throwing now? They still have the word throw in the file names despite not throwing anywhere.
The type name is determined by the constructor name of the receiver in a call site.
a0f517b to
119018c
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Landed in f33e0fc |
The type name is determined by the constructor name of the receiver in a call site. PR-URL: nodejs#58976 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The type name is determined by the constructor name of the receiver in a call site. PR-URL: nodejs#58976 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The type name is determined by the constructor name of the receiver in a call site. PR-URL: #58976 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The type name is determined by the constructor name of the receiver in a call site. PR-URL: nodejs#58976 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The type name is determined by the constructor name of the receiver in a call site. PR-URL: nodejs#58976 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The type name is determined by the constructor name of the receiver in a call site. PR-URL: #58976 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>

Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.

The type name is determined by the constructor name of the receiver in a
call site.