-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(ci): node-version to latest LTS (18) #680
Conversation
Your Render PR Server URL is https://kotti-pr-680.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-ce0d5npgp3jsllu5a2f0. |
370b384
to
e1f1ab2
Compare
e6b333a
to
a256872
Compare
fc02980
to
d72dbe1
Compare
@@ -128,7 +128,7 @@ | |||
"private": true, | |||
"repository": "git@github.com:3YOURMIND/kotti", | |||
"scripts": { | |||
"build:render.com": "yarn run watch:build-kotti-dependencies && yarn workspace @3yourmind/kotti-ui run build && yarn workspace @3yourmind/documentation run nuxt generate", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carsoli Can you explain why setting the webpack hash function wasn’t sufficient for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couldn't
d72dbe1
to
27ac926
Compare
6910c78
to
d743580
Compare
0c10d47
to
fbacf96
Compare
closing, not needed thanks to supporting node18 on #814 |
Description
PR updates node version for ci and gh-pages to latest LTS (18.12.x) on its creation date (Nov. 25th, 2022).
However:
render deployments are failing with this Error if current LTS (18.12) is used:
The particular sub-dependency that fails according to logs from
master
are webpack and its subdependencies.Given that webpack4 still uses open-ssl < 3.
Until webpack v6, this wouldn't be resolved. Therefore, we upgrade node-version for all CI jobs that use
actions/setup-node
to be 18, and set node_options to use legacy openssl like here..nvmrc
version overrides render's node-version (which is currentlylts/*
(v18).When webpack fixes this in its next major release, we can 1) upgrade 2) no longer need to opt out of using openssl3.