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

Enable json5 and json-stringify parsers by default (#92) #94

Closed
wants to merge 1 commit into from

Conversation

aspiers
Copy link

@aspiers aspiers commented Sep 19, 2021

These two parsers were accidentally omitted from the default list. In particular, this broke the choice of parser for package.json files. So add them to the default list.

This solves the original motivating factor for #92, but not #92 itself which is a request for enhanced debugging capabilities.

These two parsers were accidentally omitted from the default list.  In
particular, this broke the choice of parser for package.json files.
So add them to the default list.

This solves the original motivating factor for jscheid#92, but not jscheid#92
itself which is a request for enhanced debugging capabilities.
@jscheid
Copy link
Owner

jscheid commented Sep 19, 2021

Thanks @aspiers. See if you can figure out the failing test, otherwise I'll try and have a look soon.

@aspiers
Copy link
Author

aspiers commented Sep 20, 2021

I can't immediately, and weirdly I get a different failure when running the tests locally!

@jscheid
Copy link
Owner

jscheid commented Sep 20, 2021

See #96

@jscheid jscheid closed this Sep 20, 2021
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