Skip to content

Commit

Permalink
Merge pull request #2696 from Varying-Vagrant-Vagrants/modify/node-20…
Browse files Browse the repository at this point in the history
…-default

Switch to Node 20 to match WP core defaults
  • Loading branch information
tomjn authored Jan 3, 2024
2 parents 92263f5 + 19bd102 commit 4ba78d6
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@ permalink: /docs/en-US/changelog/

### Enhancements

* Switched the default PHP from v7.4 to v8.2 and default Node from v14 to v16 ( #2690 )
* Switched the default PHP from v7.4 to v8.2 and ~default Node from v14 to v16~ ( #2690 )
* Use Node 20 as the default version to match WP and Gutenberg ( #2696 )
* WP Coding standards v3 ( #2688 )
* VIP Coding standards v3 ( #2688 )
* Better error messages with links to docs when trying to use a PHP version that isn't installed ( #2689 )
Expand Down
6 changes: 3 additions & 3 deletions provision/core/node-nvm/provision.sh
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,9 @@ function vvv_nvm_setup() {

fi

vvv_info " - Installing Node 16 via nvm"
nvm install 16
nvm use 16
vvv_info " - Installing Node 20 via nvm"
nvm install 20
nvm use 20

vvv_info " - Ensuring vagrant user owns its own nvm folder"
chown -R vagrant:vagrant /home/vagrant/.nvm/
Expand Down

0 comments on commit 4ba78d6

Please sign in to comment.