-
Notifications
You must be signed in to change notification settings - Fork 2
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
Allow $schema
in JSON configuration file
#63
Conversation
Hi @AdamVig! |
b898227
to
e057e11
Compare
@zavoloklom Thanks for the reminder. I remember seeing this in the documentation and it just slipped my mind. All set now. |
@AdamVig Hi again! |
e057e11
to
18ac73a
Compare
@zavoloklom Hadn't noticed that CI failed, thanks for the callout. Fixed now. The CI pipeline and tooling for this project are great! |
The `$schema` property can be used by editors to provide inline documentation and validation in JSON files (see [VS Code's documentation](https://code.visualstudio.com/Docs/languages/json#_json-schemas-and-settings) as an example). Currently, the JSON schema for the `dclint` configuration file does not allow `$schema`, so adding it causes a configuration validation error and prevents the tool from starting. This adds `$schema` to the configuration JSON schema and documents how to use it.
18ac73a
to
3ffd86b
Compare
@AdamVig Thanks! Really glad to hear that. |
🎉 This PR is included in version 2.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
The
$schema
property can be used by editors to provide inlinedocumentation and validation in JSON files (see VS Code's
documentation
as an example).
Currently, the JSON schema for the
dclint
configuration file doesnot allow
$schema
, so adding it causes a configuration validationerror and prevents the tool from starting.
This adds
$schema
to the configuration JSON schema and documents howto use it.
Type of Change
Checklist