Skip to content

"Configuration file path/to/file/.csscomb.json was not found" reported when JSON is simply not valid #326

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

Closed
augbog opened this issue Oct 17, 2014 · 3 comments

Comments

@augbog
Copy link

augbog commented Oct 17, 2014

Is there a way we can make it so that if you have invalid JSON, it throws a different error message? It's not a big deal, but thought I should make a mention of it because it is kind of misleading.

Ideally, it should only throw that error if it really does not exist there at all.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/5355281-configuration-file-path-to-file-csscomb-json-was-not-found-reported-when-json-is-simply-not-valid?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).
@augbog augbog changed the title Configure file path/to/file/.csscomb.json was not found reported when JSON is simply not valid "Configuration file path/to/file/.csscomb.json was not found" reported when JSON is simply not valid Oct 17, 2014
@dmnplb
Copy link

dmnplb commented Apr 2, 2015

+1

@istro
Copy link

istro commented May 21, 2015

+1.
For me, it was a missing comma between config options generated through the builder tool on the site and "sort-order" copied from a default config file.

@ixley
Copy link

ixley commented Jun 8, 2015

+1
New to JSON and tried adding comments. Error message threw me off.

@tonyganch tonyganch added this to the Next release milestone Jun 9, 2015
haoqunjiang pushed a commit that referenced this issue Feb 14, 2016
haoqunjiang added a commit that referenced this issue Feb 14, 2016
haoqunjiang added a commit that referenced this issue Feb 15, 2016
tonyganch pushed a commit that referenced this issue Jul 11, 2016
@tonyganch tonyganch removed this from the Next release milestone Jul 11, 2016
@jdalton jdalton closed this as completed Apr 19, 2019
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

6 participants