linting, hover and... json5!

acao commented 1 year ago

linting

This is working out fine with ~json-source-map~ and json-schema-library for now!

cm6-linting

I've adopted a utility jsonPointerForPosition that made it really easy to add hover support

hover-oneOf

[x] a json-source-map parseJSONDocument replacement using the power of codemirror SyntaxNode, that adds fault tolerance and appears to be a bit faster (benchmarks to come)
[x] linting for json() mode implementations
[x] hover for json() mode implementations, with an user-configurable formatHover
[x] test coverage
[x] test CI workflow
[x] an unfortunate patch for json-schema-library (for now)
[x] a parseJSON5Document utility, and json5 support for our syntax utilities, thus, most of what is needed to enable basic json5 support!

netlify[bot] commented 1 year ago

Name	Link
Latest commit	e48c3cc7729e77209e9e2b84bde37bc95694ff02
Latest deploy log	https://app.netlify.com/sites/cm6-json-schema/deploys/64b6e274addaea0008f5824e
Deploy Preview	https://deploy-preview-2--cm6-json-schema.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

acao commented 1 year ago

@imolorhe ok, I'm merging before this gets out of hand, haha!