search

mtxr / vscode-sqltools

Database management for VSCode
https://vscode-sqltools.mteixeira.dev?utm_source=github&utm_medium=homepage-link
MIT License
1.5k stars 302 forks source link

Autodetection of Node.js runtime happens even when an override string is set in `sqltools.useNodeRuntime` #1037

Closed gjsjohnmurray closed 1 year ago

gjsjohnmurray commented 1 year ago

See https://github.com/mtxr/vscode-sqltools/issues/1024#issuecomment-1313615413

gjsjohnmurray commented 1 year ago

@GegznaV FYI I have opened this issue.

gjsjohnmurray commented 1 year ago

@GegznaV please confirm that you have not set a value for the sqltools.useNodeRuntime property at the workspace level of VS Code's settings hierarchy. Specifically, is it present in a settings.json file in a .vscode subfolder of the folder you opened in VS Code? And if your workspace is opened from a .code-workspace file please also check the JSON in that file.

GegznaV commented 1 year ago

In .vscode/settings.json there was a logical value of sqltools.useNodeRuntime setting. Now I changed it to character value and the pop-up message (warning) does not appear. I did not know that there are several settings.json files.

May "sqltools.useNodeRuntime": null mean that the default value from other settings file is used?

GegznaV commented 1 year ago

So, this issue can be closed too.

gjsjohnmurray commented 1 year ago

Just noticed that the alert gives the setting incorrectly. Should say 'sqltools.useNodeRuntime'

image