Resolve refs inside additionalItems#69
Conversation
|
Oh wow, cood catch! Thanks @KapitanOczywisty ! |
|
Just noticed this and want to point out that:
|
|
@unional When looking for |
|
Yeah, I notice that is the design of the code. The only concern is about "soundness", as language-service is used to provide runtime validation, so people writing json schema and rely on IDE to provide feedback will miss this case until they run other validation that catches this. |
|
@unional This extension provides intellisense and somewhat accurate validation, but if you want to be sure that it'll run fine on other validation tools, you need to write some tests. Here there are implemented features from different drafts and even non-standard stuff added for vscode use (error messages). I don't think that anyone is willing to bring it closer to drafts, since it's good enough for intellisense and there are very easy to use packages like ajv. |
|
Sure, no problem. And regarding drafts, really look forward for 2020-12 support. 😆 |
additionalItemswas not checked for$refs.fix microsoft/vscode#69071 fix microsoft/vscode#75394