Skip to content

Vscode 1.52.1 #2543

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

Merged
merged 14 commits into from
Feb 9, 2021
Prev Previous commit
Next Next commit
Restore Node check
Since we're following the VS Code allowed version range now to prevent
bugs use the new range instead of commenting it out.
  • Loading branch information
code-asher committed Feb 9, 2021
commit 9221efc104ee83271a7b6a8f4d9bb6cc9c48c7d0
7 changes: 0 additions & 7 deletions lib/vscode/build/npm/preinstall.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,9 @@ let err = false;

const majorNodeVersion = parseInt(/^(\d+)\./.exec(process.versions.node)[1]);

<<<<<<< HEAD
if (majorNodeVersion < 10 || majorNodeVersion >= 13) {
// We are ok building above Node 12.
// console.error('\033[1;31m*** Please use node >=10 and <=12.\033[0;0m');
// err = true;
=======
if (majorNodeVersion < 10 || majorNodeVersion >= 16) {
console.error('\033[1;31m*** Please use node >=10 and <=16.\033[0;0m');
err = true;
>>>>>>> e4a830e9b7ca039c7c70697786d29f5b6679d775
}

const cp = require('child_process');
Expand Down