sublimelsp / LSP-css

CSS, SCSS, LESS support for Sublime's LSP plugin
MIT License
37 stars 0 forks source link

Cannot override 'Unknown property' warnings #36

Closed saneef closed 3 years ago

saneef commented 3 years ago

I'm not able to override "Unknown property" warning using settings.

This is what I set inLSP-css.sublime-settings:

{
  "css.lint.validProperties": ["accent-color"],
  "scss.lint.validProperties": ["accent-color"]
}
Screenshot 2021-09-08 at 23 03 51
rwols commented 3 years ago

Install LSP-json and it’ll guide you to the misconfiguration in your LSP-css.sublime-settings

saneef commented 3 years ago

I installed LSP-json, and it's not showing any errors or warning. :(

Screenshot 2021-09-09 at 09 26 31

If you noticed any, could you point me the mistake?

jfcherng commented 3 years ago

In short, it should be

{
    "settings": {
        "css.lint.validProperties": [
            "accent-color"
        ],
        "scss.lint.validProperties": [
            "accent-color"
        ]
    }
}
jfcherng commented 3 years ago

I installed LSP-json, and it's not showing any errors or warning. :(

LSP-json doesn't provide errors for this. Weird...

saneef commented 3 years ago

Ah! 🙈

Thanks @jfcherng and @rwols!

rchl commented 3 years ago

I installed LSP-json, and it's not showing any errors or warning. :(

LSP-json doesn't provide errors for this. Weird...

I think it's a limitation of the JSON schema where when using allOf we can't also use additionalProperties: false