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

Improve errors when JSON is invalid #51

Open
orangejulius opened this issue Feb 9, 2017 · 0 comments
Open

Improve errors when JSON is invalid #51

orangejulius opened this issue Feb 9, 2017 · 0 comments

Comments

@orangejulius
Copy link
Member

Currently any issues with the pelias.json file cause the config code to output a very unclear error message that looks like this:

throw new Error( 'failed to merge config from path:' + path );

It doesn't give any indication that the problem is with the config file's syntax and definitely doesn't provide hints at the part of the file that has the syntax issue. There are many ways to validate JSON, surely we can bring one in and use it to create friendly error messages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant