You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Basically the extension vscode-eslint has a setting "eslint.runtime": "node" that is set in the workspace settings .vscode/settings.json . It means it reads the node version from the environment.
Apparently the ESLint server, or any extension, inherits the environment from the extension host. It seems there is a race condition where direnv-vscode has not updated the enviroment, meaning that e.g. eslint-vscode cannot find the node version as it is in in the PATH.
I added a question in that thread as well that maybe the extension host itself should have an event if the environment is reloaded in some sense, so that it could be emitted so that all extensions could listen for it, and restart if needed or similar. Not sure if that is a good idea or not, but throwing it out here as well.
The text was updated successfully, but these errors were encountered:
There is a long standing issue here, see microsoft/vscode-eslint#1596. I hope you have the stamina to read it.
Basically the extension vscode-eslint has a setting
"eslint.runtime": "node"
that is set in the workspace settings.vscode/settings.json
. It means it reads thenode
version from the environment.We load the node version via direnv. I think this comment best sums it up: microsoft/vscode-eslint#1596 (comment)
Apparently the ESLint server, or any extension, inherits the environment from the extension host. It seems there is a race condition where direnv-vscode has not updated the enviroment, meaning that e.g. eslint-vscode cannot find the
node
version as it is in in thePATH
.I added a question in that thread as well that maybe the extension host itself should have an event if the environment is reloaded in some sense, so that it could be emitted so that all extensions could listen for it, and restart if needed or similar. Not sure if that is a good idea or not, but throwing it out here as well.
The text was updated successfully, but these errors were encountered: