doc: Update README.md

[beliaev-maksim]

Update installation instruction

[gengjiawen]

thx for the contribution

[karlhorky]

@gengjiawen in comment 1161506873: Can you delete this line ? I think for most user, this is not necessary.

@gengjiawen some of our students with Windows machines have problems with running this npm config set msvs_version 2017 command as part of our System Setup guide for Windows, so it's interesting to us that you removed it with the explanation that "it's not necessary for most users".

Their problems look like upleveled/system-setup#31:

PS C:\Windows\system32> npm config set msvs_version 2017
npm ERR! `msvs_version` is not a valid npm option

npm ERR! A complete log of this run can be found in: C:\Users\khayo\AppData\Local\npm-cache\_logs\2023-08-22T12_12_13_960Z-debug-0.log

Can you expand on the explanation why it's not necessary for most users? Which users would it be necessary for?

Our Windows System Setup Guide suggests to install the Chocolatey packages visualstudio2017-workload-vctools and python and then configure msvs_version, which leads to this failure.

If you have any suggestions or remarks, we're happy to also do the setup a different way - but it should be able to be performed via the command line, no GUI tools or downloads from websites.

[gengjiawen]

Can you expand on the explanation why it's not necessary for most users?

Gyp can automatically choose msvs version.

Which users would it be necessary for?

You have a project only can compile in vs2017, but tbh MS don't maintain old visual studio.

My general advice: Install visual studio and Python on windows is only needed for node-gyp.

[karlhorky]

Ok great, then if I'm understanding correctly, @ProchaLu will try to upgrade to visualstudio2022-workload-vctools ("Visual C++ build tools workload for Visual Studio 2022 Build Tools 1.0.0") and remove the npm config set msvs_version ... command to see if this works, thanks!

• Fix npm msvs_version error in Windows system-setup guide upleveled/system-setup#31 (comment)