Skip to content
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

incompatible upstream: vscode-json-languageservice 3.10.0 #345

Closed
3 tasks done
bollwyvl opened this issue Nov 4, 2020 · 3 comments
Closed
3 tasks done

incompatible upstream: vscode-json-languageservice 3.10.0 #345

bollwyvl opened this issue Nov 4, 2020 · 3 comments

Comments

@bollwyvl
Copy link
Contributor

bollwyvl commented Nov 4, 2020

Describe the bug

[email protected] dropped a few hours ago, and has removed src/services/jsonDefinition.ts.

Thanks for maintaining this, hope this can help save some pain!

Expected Behavior

Should run normally.

Current Behavior

Does not run.

Steps to Reproduce

  1. $> npm install yaml-language-server
$> node ./node_modules/.bin/yaml-language-server || echo "💥"

internal/modules/cjs/loader.js:955
  throw err;
  ^

Error: Cannot find module 'vscode-json-languageservice/lib/umd/services/jsonDefinition'
Require stack:
- ~/node_modules/yaml-language-server/out/server/src/languageservice/services/yamlDefinition.js
- ~/node_modules/yaml-language-server/out/server/src/languageservice/yamlLanguageService.js
- ~/node_modules/yaml-language-server/out/server/src/server.js
- ~/node_modules/yaml-language-server/bin/yaml-language-server
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:952:15)
    at Function.Module._load (internal/modules/cjs/loader.js:835:27)
    at Module.require (internal/modules/cjs/loader.js:1012:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (~/node_modules/yaml-language-server/out/server/src/languageservice/services/yamlDefinition.js:5:26)
    at Module._compile (internal/modules/cjs/loader.js:1123:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1143:10)
    at Module.load (internal/modules/cjs/loader.js:972:32)
    at Function.Module._load (internal/modules/cjs/loader.js:872:14)
    at Module.require (internal/modules/cjs/loader.js:1012:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '~/node_modules/yaml-language-server/out/server/src/languageservice/services/yamlDefinition.js',
    '~/node_modules/yaml-language-server/out/server/src/languageservice/yamlLanguageService.js',
    '~/node_modules/yaml-language-server/out/server/src/server.js',
    '~/node_modules/yaml-language-server/bin/yaml-language-server'
  ]
}
💥

Environment

  • Windows
  • Mac
  • Linux
@adrianlzt
Copy link

adrianlzt commented Nov 12, 2020

Workaround for users of coc:

cd $HOME/.config/coc/extensions/node_modules/coc-yaml/node_modules/yaml-language-server
npm install [email protected]

@JPinkney
Copy link
Contributor

This is only when you use npm right?

If I do:

yarn
yarn run compile

everything is done successfully

@JPinkney
Copy link
Contributor

I've merged the PR to make it compatible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants