lexical-lsp / vscode-lexical

VSCode extension for Lexical, the modern Elixir language server
Apache License 2.0
36 stars 6 forks source link

chore: Migrate to npm #73

Closed Blond11516 closed 7 months ago

Blond11516 commented 7 months ago

While I usually find yarn a little easier to use, @vscode/vsce only supports yarn version 1, which does not properly handle the engines field from package.json. This in turn causes a test failure with the latest version of glob. I don't think being blocked on a depedency upgrade (and potentially others in the future) is worth sticking with yarn.

Blond11516 commented 7 months ago

Turns out we didn't even use glob (nor ts-node for that matter) so the migration wasn't strictly necessary. That said I still think it's worth it because it lets us use a package manager that is actively maintained and should offer better compatibility with the JS ecosystem moving forward.