Skip to content
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

Fix: formatting in prepareStackTrace when error isn't of instance Error #237

Merged
merged 2 commits into from
Apr 8, 2019

Conversation

legraphista
Copy link
Contributor

@legraphista legraphista commented Mar 23, 2019

console.trace sends an object with name and message instead of an Error,
thus rendering the function unusable due to erroneous message output of
"[object Object]". This fixes #195

console.trace sends on object with name and message instead of an Error,
thus rendering the function unusable due to erroneous message output of
"[object Object]". This fixes evanw#195
@LinusU
Copy link
Collaborator

LinusU commented Apr 2, 2019

Thanks for submitting this PR, it looks great!

Would you mind adding a test case that calls console.trace? Thanks!

@legraphista
Copy link
Contributor Author

Hi!

Sorry for the delay, I've added a test of console.trace which fails on master but passes on the PR branch.

@LinusU
Copy link
Collaborator

LinusU commented Apr 8, 2019

Perfect, thanks!

@LinusU LinusU merged commit f2a0fbf into evanw:master Apr 8, 2019
@LinusU
Copy link
Collaborator

LinusU commented Apr 8, 2019

Released as 0.5.12 🚀

@legraphista legraphista deleted the fix/195 branch April 11, 2019 09:07
@legraphista legraphista restored the fix/195 branch April 11, 2019 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using console.trace with value outputs [object Object]
2 participants