-
Notifications
You must be signed in to change notification settings - Fork 36
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
test(language-server): Use vitest intead of mocha #1727
Conversation
🤖 Pull request artifactsempty string
|
What do you think about this warning?
|
Seems like a sign to move over to esbuild (which would also let us migrate to pnpm) but we'll have to stick to mocha/cjs for e2e tests |
99312ec
to
8c51c69
Compare
@Druue Nice find, I'm ok if we get stuck on |
Only language-server package for now
I think the warning comes from |
Update: warning is gone |
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.
Nice!
Only language-server package for now