Updated Node version to 22.14.0#543
Conversation
services closer together.
|
Just letting this sit here for a bit while i cool off |
| @@ -1,7 +1,8 @@ | |||
| FROM node:18.17.0 as builder | |||
| FROM node:22.14.0 as builder | |||
There was a problem hiding this comment.
I wonder if we can do something about this being a variable? I'm just trying to remove any hard coded references to the version so if we update it again we have minimal places to update.
Would just using lts be good enough I wonder?
NODE_VERSION file to grab the correct node version.
the setup_node action in the github actions
NODE_VERSION file
| :Init | ||
| set SCRIPT_DIR=%~dp0 | ||
| set NODE_VERSION=v18.17.0 | ||
| set /p NODE_VERSION=<"%SCRIPT_DIR%/../../../NODE_VERSION" |
There was a problem hiding this comment.
Sanity check on the < in this changed file and the one above. That is intentional, yes? (What does it do?)
There was a problem hiding this comment.
They're redirection operators that are reading the file contents. The opposite of
echo "line" > file.txt
|
Good call on using the NODE_VERSION file! |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
[UE5.5] Merge pull request #543 from Belchy06/version
3 participants
Relevant components:
Problem statement:
Warnings are currently being thrown during
npm installregarding an outdated version of Node.Solution
Bump the Node version used in this repo to that latest LTS: 22.14.0