Closed imolorhe closed 4 months ago
Latest commit: f5b1ebfbd76e0dfab13243b7d68d985045aa6936
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Name | Link |
---|---|
Latest commit | f5b1ebfbd76e0dfab13243b7d68d985045aa6936 |
Latest deploy log | https://app.netlify.com/sites/codemirror-json-schema/deploys/65d7fb736df40d0008720113 |
Deploy Preview | https://deploy-preview-85--codemirror-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.
oh this is BEAUTIFUL
Closes #84
To support YAML (and potentially other formats), I updated the logic to "resolve" tokens before using them. This is important since the YAML grammar/tokens is different from the JSON grammar/tokens (which is very similar to the JSON5 grammar). So we map the tokens from the different grammars to the equivalent token in the "base" grammar (currently this is the JSON grammar tokens but I think we can/should map to some other arbitrary token set which will force us to properly implement the mappings, otherwise additional changes made may work in JSON mode but without proper testing, may not work in YAML mode, if the mappings were not added).