Skip to content

Conversation

@connorjclark
Copy link
Contributor

Continuing #205

The tests were failing due to not falling back to the original name when the source map position did not have a name.

], [
'Error: test',
/^ at Object\.exports\.test \((?:.*[/\\])?.generated.js:1:34\)$/
/^ at Object\.exports\.test \((?:.*[/\\])?\.generated.js:1:34\)$/
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are various minor regex changes too

], [
'Error: test',
/^ at Object\.exports\.test \((?:.*[/\\])?.generated.js:1:34\)$/
/^ at Object\.exports\.test \((?:.*[/\\])?\.generated.js:1:34\)$/
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@LinusU
Copy link
Collaborator

LinusU commented Aug 9, 2018

Looks great! Just waiting for the CI to turn green 👍

@LinusU LinusU merged commit 21c4dc1 into evanw:master Aug 9, 2018
@LinusU
Copy link
Collaborator

LinusU commented Aug 9, 2018

Published as v0.5.8 🚢

Thanks 🎉

@connorjclark
Copy link
Contributor Author

Awesome - thanks for the quick release!

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.

2 participants