-
-
Notifications
You must be signed in to change notification settings - Fork 13.3k
vscode-eslint-language-server v3.0.20 (new formula) #259385
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
base: main
Are you sure you want to change the base?
Conversation
9ed8f94 to
8872123
Compare
8872123 to
569b1a5
Compare
de89d10 to
75f7b49
Compare
| class VscodeEslintLanguageServer < Formula | ||
| desc "ESLint language server from VSCode" | ||
| homepage "https://github.com/microsoft/vscode-eslint" | ||
| url "https://github.com/microsoft/vscode-eslint/archive/refs/tags/release/3.0.20.tar.gz" |
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.
using npm registery?
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.
Sorry, please clarify. There is no NPM package
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.
ah, I thought it should have some npm package for this to install, let me run another check
| system "npm", "ci" | ||
| system "npm", "run", "compile:server" | ||
|
|
||
| libexec.install Dir["*"] | ||
|
|
||
| # Create a wrapper script that explicitly invokes node with the server script | ||
| (bin/"vscode-eslint-language-server").write <<~EOS | ||
| #!/usr/bin/env bash | ||
| exec "#{Formula["node"].opt_bin}/node" "#{libexec}/server/out/eslintServer.js" "$@" | ||
| EOS |
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.
check out other node formulae
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.
Please elaborate a bit more, vscode-eslint does not provide a binary by default so I think we have to create a wrapper script like this
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.
can you provide the current upstream installation methods?
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.
Other than than of what is mentioned in the README there are none I believe. The language server is bundled together with the vscode plugin and not provided separately and majority of people just click the install button in vscode.
There is another brew package here that also tries to address this issue by repackaging the language servers from vscode as a separate NPM package. However, ideally I would not like to have to rely on this one as already it seems like it's not being updated and is not controlled by the vscode maintainers.
What I'm trying to do for this formula is the same way the Arch package maintainers are doing here. (Cloning the official repo, compiling the server and providing it as a standalone binary for editors other than vscode which rely on LSPs)
a5a0a79 to
b1a71fe
Compare
… Server Add a new formula to provide the Language Server Protocol implementation for ESLint from the official vscode-eslint repository. This allows editor integration with ESLint through LSP-compatible editors. Co-authored-by: Rui Chen <rui@chenrui.dev>
b1a71fe to
ddda1e2
Compare
Description
Adds a new formula for vscode-eslint-language-server
Verification
brew install --build-from-source vscode-eslint-language-server.rbbrew audit --strict --new vscode-eslint-language-server.rbbrew test vscode-eslint-language-server.rbDetails