Skip to content

Update nvmrc to node 8#67183

Merged
joaomoreno merged 1 commit intomicrosoft:masterfrom
thien-do:patch-1
Jan 28, 2019
Merged

Update nvmrc to node 8#67183
joaomoreno merged 1 commit intomicrosoft:masterfrom
thien-do:patch-1

Conversation

@thien-do
Copy link
Contributor

As noted in the wiki, node's version should be larger than 8.12 so 8.9.2 will not work.

In fact, I think we only need to define the target version as 8 so we can have the latest version that satisfies the requirement (>= 8.12.0, < 9.0.0). At the time of writing it is 8.15 which works perfectly fine (installed + watch + run successfully).

As noted in [the wiki](https://github.com/Microsoft/vscode/wiki/How-to-Contribute#prerequisites) node's version should be larger than 8.12 so 8.9.2 will not work.

In fact, I think we only need to define the target version as `8` so we can have the latest version that satisfies the requirement (`>= 8.12.0, < 9.0.0`). At the time of writing it is `8.15` which works perfectly fine (install + watch + run).
@joaomoreno joaomoreno merged commit b1bf810 into microsoft:master Jan 28, 2019
@joaomoreno joaomoreno added this to the December/January 2019 milestone Jan 28, 2019
@joaomoreno joaomoreno added the engineering VS Code - Build / issue tracking / etc. label Jan 28, 2019
@joaomoreno
Copy link
Member

Thanks!

@thien-do thien-do deleted the patch-1 branch January 28, 2019 08:19
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

engineering VS Code - Build / issue tracking / etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants