Skip to content
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

Support Node 18 #3699

Closed
wants to merge 4 commits into from
Closed

Support Node 18 #3699

wants to merge 4 commits into from

Conversation

sneridagh
Copy link
Member

@sneridagh sneridagh commented Sep 29, 2022

It will be LTS on 2022-10-25, better get prepared.

It seems to me that it's a bit cumbersome that time of the year that we support up to 3 Node versions (Oct-Apr) when 14 will be deprecated... it's CI time nonetheless...

Maybe something we can talk during the conference. /cc @plone/volto-team

@netlify
Copy link

netlify bot commented Sep 29, 2022

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit 29f57b8
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/634fcc0abee8d700084dced5

@cypress
Copy link

cypress bot commented Sep 29, 2022



Test summary

13 0 0 0


Run details

Project Volto
Status Passed
Commit 29f57b8
Started Oct 19, 2022 10:12 AM
Ended Oct 19, 2022 10:15 AM
Duration 02:40 💡
OS Linux Ubuntu - 20.04
Browser Chrome 106

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@nzambello
Copy link
Member

I'm using Volto on node 18 for a while, didn't notice any issue

@nzambello
Copy link
Member

Tests failing ref: webpack/webpack#14532

nzambello and others added 3 commits October 15, 2022 11:18
* master: (23 commits)
  The listing block icon has been improved to avoid confusions with the normal text list (#3785)
  Fix autocomplete widget with an empty search result (#3757)
  Back to development
  Release 16.0.0-alpha.43
  Prepare for release
  Upgrade dependency rrule (optional dependency luxon removed) (#3781)
  Fix logout to stay on the same page where the user was (#3739)
  Object browser: image search should only show images (#3738)
  Update CHANGELOG.md
  Update changelog
  Update branch, docs location
  add dutch translations
  Trigger on 6-dev
  Fix spelling
  Add action to trigger Plone documentation update
  Fixed typo for ordinal number at spanish translation
  Fixed some typos for spanish translation
  Fixed a msg string translation
  Fixed a msg string translation
  Updated spanish translation
  ...
@sneridagh
Copy link
Member Author

sneridagh commented Oct 19, 2022

This is pitiful: webpack/webpack#14532 (comment)

I investigated it a bit. It seems that they won't fix Webpack 4. Either we go full length and upgrade to 5, or we cope with NODE_OPTIONS=--openssl-legacy-provider solution... :(

Great, and it seems that the option is invalid for NodeJS < 18 ...

Node 18 is LTS in 2022-10-25 :/

/cc @plone/volto-team @davisagli

@davisagli
Copy link
Member

@sneridagh I guess we need both: for the short term, add NODE_OPTIONS=--openssl-legacy-provider for Node 18 only, and in the long term, we do need to update to a modern webpack version sometime. It should be easier now that we have Razzle 4, which supports webpack 5. But it'll take a bit of updating our webpack plugins.

@tiberiuichim tiberiuichim added this to the 17.x.x milestone Nov 26, 2022
@davisagli
Copy link
Member

We added support for Node 18 in #4086 instead.

@davisagli davisagli closed this Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants