-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
stylelint.validate
in README doesn’t match with that in v1.0.0 Breaking change
#268
Comments
Good catch! Updated the readme in 4464251. |
but my vscode can’t lint less file in v1.0.0, until i set |
Looks like there's a mistake in the extension: all three languages are correctly specified in the server, but Less is missing from the extension manifest. I'll push up a patch update in a moment. Thank you again! |
@izumiiAoba I just pushed up v1.0.1, which should fix this problem. If it doesn't, let me know and I'll be happy to help! |
it work nice, thanks a lot 😎 |
@adalinesimonian Stylelint doesn't support Less out-of-the-box (a custom syntax is needed). Only CSS (and PostCSS because it's CSS-compatible) are. So, I believe the Sorry that I missed this the first time around. |
@jeddy3 Huh! I wonder where Less snuck in, then. When I originally threw Less into the defaults, I was certain I had seen something about the defaults needing to be CSS, Less, and PostCSS, which is why I kept Less in the server defaults when preparing for v1/14. However, I went back and pulled up #233 and I don't see any mention of Less, either now or in a past edit of a comment. So I'm honestly not sure where it came from. So should we pull it out of defaults, push out another patch, regretfully inform our bug-reporting friend here that we goofed up? |
Yes, I think so. |
README/Extension Settings
v1.0.0 changelog/Breaking changes
less is not set by default
The text was updated successfully, but these errors were encountered: