stacktracejs / error-stack-parser

Extract meaning from JS Errors
https://www.stacktracejs.com/#!/docs/error-stack-parser
MIT License
450 stars 52 forks source link

Add a failling test for node paths with parens #61

Open mattwynne opened 3 years ago

mattwynne commented 3 years ago

Description

See https://github.com/stacktracejs/error-stack-parser/issues/62

This PR is just a failing test to reproduce the problem. I'm hoping it should help someone who's a newcomer to open source and wants to contribute a fix.

I expect we can close this PR without merging it when someone submits a better PR that includes a fix 😄

Motivation and Context

Failing test to reproduce https://github.com/stacktracejs/error-stack-parser/issues/62

How Has This Been Tested?

Added a new test case in spec/error-stack-parser-spec.js for this scenario.

Types of changes

Checklist:

mattwynne commented 3 years ago

I don't know who needs to hear this but I have no node_modules/.bin/jscs in my local project. Is this step in the template out of date?

niftylettuce commented 2 years ago

If this is still an issue, can you resolve the conflict and ping me back once done?

mattwynne commented 2 years ago

We ended up solving the problem by switching to stackchain so I've lost personal interest in this I'm afraid.