VACMS-24047: Updates nvm to 22#24211
Conversation
|
Checking composer.lock changes... |
There was a problem hiding this comment.
Pull request overview
This PR updates the bootstrap script that installs Node via nvm, with the apparent goal of moving local/container setup toward the repository’s current Node 22 baseline. In this codebase, that script is used by DDEV and Tugboat startup flows.
Changes:
- Changes
scripts/install-nvm.shto install Node22.13.0instead of16.19.1. - Aligns that script with the current root
.nvmrcvalue. - Leaves the rest of the repository’s Node version pins unchanged.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion" # This loads nvm bash_completion | ||
|
|
||
| nvm install 16.19.1 | ||
| nvm install 22.13.0 |
| [ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion" # This loads nvm bash_completion | ||
|
|
||
| nvm install 16.19.1 | ||
| nvm install 22.13.0 |
|
Tugboat has finished building the preview for this pull request! Links:
Dashboard: |
Cypress Accessibility Violations
|
Description
Relates to #24047
Generated description
(Select this text, hit the Copilot button, and select "Generate".)
Testing done
Screenshots
QA steps
What needs to be checked to prove this works?
What needs to be checked to prove it didn't break any related things?
What variations of circumstances (users, actions, values) need to be checked?
As user uid with user_role
Definition of Done
Select Team for PR review
CMS TeamPublic websitesFacilitiesUser supportAccelerated PublishingIs this PR blocked by another PR?
DO NOT MERGEDoes this PR need review from a Product Owner
Needs PO reviewCMS user-facing announcement
Is an announcement needed to let editors know of this change?