microsoft / inshellisense

IDE style command line auto complete
MIT License
8.24k stars 181 forks source link

Tryin to use it with nvm on Windows throws "SyntaxError: Unexpected reserved word" #17

Closed arekgotfryd closed 8 months ago

arekgotfryd commented 8 months ago

Describe the bug Tryin to use it with nvm on Windows throws: file:///C:/Users/User/AppData/Roaming/nvm/v12.22.12/node_modules/@microsoft/inshellisense/build/index.js:13 .version(await getVersion(), "-v, --version", "output the current version") ^^^^^

SyntaxError: Unexpected reserved word ←[90m at Loader.moduleStrategy (internal/modules/esm/translators.js:140:18)←[39m ←[90m at async link (internal/modules/esm/module_job.js:42:21)←[39m

To Reproduce Steps to reproduce the behavior:

  1. Open cmd
  2. Install npm install -g @microsoft/inshellisense
  3. Run inshellisense bind
  4. See error

Expected behavior I expect to not get an error

Environment

kamranrad1993 commented 8 months ago

you are using outdated node version.

sudo npm install -g n sudo n lts

arekgotfryd commented 8 months ago

Could you please specify which versions are supported in the Readme?

yCodeTech commented 8 months ago

you are using outdated node version.

sudo npm install -g n sudo n lts

Windows doesn't have the sudo command....

cpendery commented 8 months ago

you are using outdated node version. sudo npm install -g n sudo n lts

Windows doesn't have the sudo command....

@ycodetech I've found this guide helpful for managing node versions on Windows & here is the installer for the latest Node LTS which supports Windows. inshellisense only supports Node >= 16.X

yCodeTech commented 8 months ago

you are using outdated node version. sudo npm install -g n sudo n lts

Windows doesn't have the sudo command....

@ycodetech I've found this guide helpful for managing node versions on Windows & here is the installer for the latest Node LTS which supports Windows. inshellisense only supports Node >= 16.X

Oh I know. I've had Node installed for a year or so, and l have no problems using it. I was merely stating the fact the comment using sudo is incorrect for Windows as it doesn't have the command internally.