-
Notifications
You must be signed in to change notification settings - Fork 215
Add Tailwind CSS language mode tests #1216
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
Conversation
32d1e17
to
6239d03
Compare
8009fbc
to
f65649f
Compare
0a89777
to
58e0be9
Compare
f65649f
to
fceef0c
Compare
58e0be9
to
fde6a28
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not 100% sure how to test it properly apart from running it locally in vscode and it seems to work as expected.
Also looks like most of it is moving things around + cleanup.
Additional tests are also nice and no older tests break so that also a win.
}) | ||
|
||
connection.onInitialize((params: InitializeParams) => { | ||
workspaceFolders = (<any>params).workspaceFolders |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think I've ever used a cast like that in TypeScript ever haha. Think I always do this:
workspaceFolders = (<any>params).workspaceFolders | |
workspaceFolders = (params as any).workspaceFolders |
Just a comment, no change required though.
fde6a28
to
a56c127
Compare
This PR does a small bit of refactoring on the bundled CSS language server and adds tests for it to ensure we don't break stuff