Remove TODO in JsonLspAdapter (#15338)
Some checks are pending
CI / Check formatting and spelling (push) Waiting to run
CI / (macOS) Run Clippy and tests (push) Waiting to run
CI / (Linux) Run Clippy and tests (push) Waiting to run
CI / (Windows) Run Clippy and tests (push) Waiting to run
CI / Create a macOS bundle (push) Blocked by required conditions
CI / Create a Linux bundle (push) Blocked by required conditions
CI / Create arm64 Linux bundle (push) Blocked by required conditions
Deploy Docs / Deploy Docs (push) Waiting to run

As the post-install issue is fixed via #15331 , we can remove this
`TODO` now. I have tested on win11, it works fine.

Release Notes:

- N/A
This commit is contained in:
张小白 2024-07-28 01:05:48 +08:00 committed by GitHub
parent f1d777434b
commit f88278111e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -166,11 +166,9 @@ impl LspAdapter for JsonLspAdapter {
.await;
if should_install_language_server {
// TODO: the postinstall fails on Windows
self.node
.npm_install_packages(&container_dir, &[(package_name, latest_version.as_str())])
.await
.log_err();
.await?;
}
Ok(LanguageServerBinary {