chore: Add npm update to build script #6656
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since #6629, npm 6 in the minimum version, so install it at setup time. Really this installs the latest version because I didn't want to deal with adding yet another 6 somewhere. If that's too dangerous we can change this to be explicitly 6–let me know.
This threw me because I created a PR and ran the app with
npm
5 and received no error messages. It wasn't until my noisy PR came in that @gziolo pointed out I was running an out-of-datenpm
.Description
Globally install
npm
during set up.How has this been tested?
Ran the script on a new machine; it worked.
Types of changes
Add
npm install npm -g
to start up.Checklist: