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

Provide source context with error #5

Merged
merged 2 commits into from
Dec 9, 2016
Merged

Provide source context with error #5

merged 2 commits into from
Dec 9, 2016

Conversation

nikku
Copy link
Contributor

@nikku nikku commented Dec 8, 2016

With these changes we provide proper context for errors:

ERROR in ./src/header.html
Unexpected token (14:23)
12:     },
13:     computed:
14:       salutation(dude) {
                           ^
15:         return hello(dude);
16:       }
 @ ./src/counter.html 3:0-30
 @ ./src/index.js
 @ multi index

Before errors were shown with their message only and excluded the frame data.

Allow webpack to print verbose error logs,
including the error context (in case of parse errors).
* add test verifying that we properly throw validation errors
* restructure test suite
@nikku nikku merged commit 119b619 into master Dec 9, 2016
@nikku nikku deleted the error-context branch December 9, 2016 10:34
rixo added a commit to rixo/svelte-loader that referenced this pull request Jan 14, 2021
…h-4.17.19

Bump lodash from 4.17.15 to 4.17.19
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.

1 participant