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.

Sign up for GitHub

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

Jump to bottom

Provide readable error messages on top of Uglify #2641

Closed
gaearon opened this issue Jun 27, 2017 · 3 comments
Closed

Provide readable error messages on top of Uglify #2641

gaearon opened this issue Jun 27, 2017 · 3 comments
Labels
contributions: claimed difficulty: complex issue: proposal

Comments

@gaearon
Copy link
Contributor

gaearon commented Jun 27, 2017 •
edited
Loading

Not sure if we can even do that, but the way it prints parsing errors is currently completely non-obvious:

Unexpected token: name (i) [./~/node-thermal-printer/node-thermal-printer.js:403,0][static/js/main.47fb6eaa.js:20938,12]

Any improvement here would be nice.
@viankakrisna viankakrisna mentioned this issue Jun 27, 2017
Merged
@gaearon
Copy link
Contributor Author

gaearon commented Jun 27, 2017

Woah. #2650 looks sweet!

@gregglind
Copy link

(the gist of that error was that ES6 code wasn't being transpiled properly, and the fix is to move it into /src)

@Timer
Copy link
Contributor

Timer commented Aug 3, 2017

#2650 has been merged.

@Timer Timer closed this as completed Aug 3, 2017
@lock lock bot locked and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
contributions: claimed difficulty: complex issue: proposal
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gregglind @Timer @gaearon