-
Notifications
You must be signed in to change notification settings - Fork 156
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
add the schema link in toml files #2104
Conversation
8f26262
to
d80c901
Compare
c12d0ec
to
674b302
Compare
create & add a schema to graveyard.tom tag kavita with "paid-content" aaaaa the autoformater tag kavita with "paid-content" aaaaa the autoformater add the schema link in toml files create & add a schema to antifeatures.toml add 'draft' property to the wishlist schema
… into add-schema-link-toml
Are those schemas used by linter tools? Do we have a CI job that used them before merging or do we need one? |
yes it's used by an automatic linter, but only for Lines 14 to 19 in 6874158
it's also used by my linter in my IDE, so i can easilly spot errors like:
and fix them by hand even if they're not used by an automatic CI, it's a good enhancement i think:
|
I think too, I'm interested about how to automatized it and adding it where it's useful like in some CI jobs or equivalent |
notes:
graveyard.toml
does not have a schema -> fixedantifeatures.toml
does not have a schema -> fixedAdditional properties are not allowed ('draft' was unexpected)
errors forwishlist.toml
-> fixedPR status: ready