jsonlint icon indicating copy to clipboard operation
jsonlint copied to clipboard

[Feature Request] More helpful error message when single quotes detected

Open agilgur5 opened this issue 6 years ago • 2 comments

If you use single quotes, it will (confusingly) tell you "expected STRING, got undefined". Single quotes are in fact strings, but invalid JSON, and therefore a very common error (see #46 , and https://github.com/getinsomnia/insomnia/issues/1150), so I think a more informative error message would be much more helpful.

agilgur5 avatar Jan 05 '19 23:01 agilgur5

You could add single quotes to the lexer and instead of recognising them as string declarators, you would just make the tokenisation fail. I did something similar for JavaScript-like comments, so that they could be either ignored, or reported by a better error message.

prantlf avatar May 18 '19 20:05 prantlf

Oh cool, thanks for the suggestion @prantlf !

Good luck on the fork and hope it does well! I also maintain a heavily updated fork of a library :)

agilgur5 avatar May 19 '19 19:05 agilgur5