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

Test: ESLint #11100

Closed
2 tasks done
dbaeumer opened this issue Aug 29, 2016 · 3 comments
Closed
2 tasks done

Test: ESLint #11100

dbaeumer opened this issue Aug 29, 2016 · 3 comments
Assignees
Milestone

Comments

@dbaeumer
Copy link
Member

dbaeumer commented Aug 29, 2016

Test for microsoft/vscode-languageserver-node#79
Complexity: 3

The module resolving algorithm to load the eslint library has been re-implemented. It now looks for the eslint module in the parent folder of the workspace folder as well. Frist install this ESLint extension
vscode-eslint-1.0.1.zip as a side extension (Note you have to the zip to vsix). Then test from a workspace folder:

  • that the eslint is correctly loaded from the workspace folder or any parent folder.
  • that the eslint library is correctly loaded from a global npm location as well.
  • install eslint lib at a custom location and point the setting eslint.nodePath value to it (if installed in a node_modules folder the node_modules name must be part of the path). Ensure that the eslint library can be loaded.
  • install an eslint plugin and configure it in the .eslintrc.json file accordingly. Ensure that the eslint plugin gets loaded correctly as well as long as it is installed at the same 'location' (e.g. node_modules folder) as eslint library itself.
@isidorn
Copy link
Contributor

isidorn commented Aug 30, 2016

@dbaeumer I would suggest this item is tackled by one more tester

@roblourens
Copy link
Member

I tried setting eslint.nodePath to .../node_modules/eslint and .../node_modules/.bin/eslint before setting it to .../node_modules. I don't know if there's some way it could be flexible enough to work with the first one at least?

@dbaeumer
Copy link
Member Author

@roblourens thanks for testing it. the nodePath variable works like the NODE_PATH env variable. So the correct path in your example would have been ../node_modules without the eslint. The reason being is that eslint could refer to plugins which are installed side by side with eslint itself.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
@roblourens roblourens removed their assignment Oct 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants