palantir / python-language-server

An implementation of the Language Server Protocol for Python
MIT License
2.6k stars 282 forks source link

Unable to run pyls with VS code #834

Open CHRC-Int opened 4 years ago

CHRC-Int commented 4 years ago

Hello

I'm trying to get develop this against VS code but when I run yarn install I'm warned that

yarn install v1.22.4
[1/5] Validating package.json...
warning pyls@0.0.1: The engine "vscode" appears to be invalid.
[2/5] Resolving packages...
success Already up-to-date.
$ node ./node_modules/vscode/bin/install
Detected VS Code engine version: ^1.15.0
Found minimal version that qualifies engine range: 1.15.0
Fetching vscode.d.ts from: https://raw.githubusercontent.com/Microsoft/vscode/8b95971d8cccd3afd86b35d4a0e098c189294ff2/src/vs/vscode.d.ts
vscode.d.ts successfully installed!

and if I later try yarn run vscode I get

yarn run v1.22.4
warning pyls@0.0.1: The engine "vscode" appears to be invalid.
$ npm run vscode:prepublish && VSCODE=$(which code-insiders || which code || echo echo ERROR: neither the code nor code-insiders vscode executable is installed); USER=dummy-dont-share-vscode-instance $VSCODE --user-data-dir=$PWD/.vscode-dev/user-data --extensionHomePath=$PWD/.vscode-dev/extensions --extensionDevelopmentPath=$PWD $* ..
npm WARN lifecycle The node binary used for scripts is /tmp/yarn--1595363275707-0.8429611618286397/node but npm is using /usr/bin/node itself. Use the `--scripts-prepend-node-path` option to include the path for the node binary npm was executed with.

> pyls@0.0.1 vscode:prepublish /home/cesally/python-language-server/vscode-client
> tsc -p ./

/bin/sh: 1: /mnt/c/Users/cchak/AppData/Local/Programs/Microsoft: not found
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Any help resolving this would be appreciated.

ccordoba12 commented 4 years ago

No one is maintaining the VSCode extension at the moment, nor we have plans to fix that either, sorry (see my https://github.com/palantir/python-language-server/issues/194#issuecomment-646316798 about this).

However, you're welcome to submit a pull request to improve the situation and we'll be happy to merge it.

CHRC-Int commented 4 years ago

Ok; thanks