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

future: support Node 18 #11220

Closed
paul-marechal opened this issue May 30, 2022 · 9 comments · Fixed by #12711
Closed

future: support Node 18 #11220

paul-marechal opened this issue May 30, 2022 · 9 comments · Fixed by #12711
Labels
ci issues related to CI / tests

Comments

@paul-marechal
Copy link
Member

paul-marechal commented May 30, 2022

While dropping support for Node 12 in #11210 I tried adding testing for Node 18 but failed.

It seems like there's a couple breakages in Node 18 that we will need to adapt to once we decide to support it:

  • It needs Ubuntu >= 20.04 to run because it uses a newer version of glibc.
  • net.Server.listen(port, address, ...) now defaults to an IPv6 localhost address when unspecified.
  • The macaddress npm package used by plugin-ext breaks on Node 18.
  • We need to update the macaddress package for it to work on Node 18.
  • More?
@paul-marechal paul-marechal added the ci issues related to CI / tests label May 30, 2022
@Alexander-Taran
Copy link
Contributor

pull request to fix macaddress package
scravy/node-macaddress#40

@Alexander-Taran
Copy link
Contributor

Alexander-Taran commented Jun 23, 2022

Fixed macaddress is published version to npmjs

@anavarre
Copy link

Any update on Node.js 18 support? It's been a year since this issue was raised and with the upcoming EOL of Node.js 16 in September I'd like to make sure that we have a secure option then. Thanks!

@shriacquia
Copy link

Any update on Node.js 18 support?

@paul-marechal
Copy link
Member Author

Gave it a quick try and it seems trivial to bump to node 18.x so I opened a PR for it.

Please try the PR and tell us if you find any regressions that weren't caught by our CI.

@vince-fugnitto
Copy link
Member

Please note that we generally want to follow the same runtime present in VS Code for compatibility with extensions.
They declare that versions higher than 17 have not yet been tested, and still use 16 during CI:

@paul-marechal
Copy link
Member Author

We use Electron 23 which embarks Node 18 already.

@danepowell
Copy link
Contributor

Since this seems to be blocked on vscode, note that vscode will move to Node 18 as part of Electron 25: microsoft/vscode#184021

@paul-marechal
Copy link
Member Author

paul-marechal commented Aug 8, 2023

I don't think (theia) moving to node 18 is "blocked on VS Code" as our current version of Electron runs node 18.

edit: Unless you meant that the move is blocked on the VS Code side, not ours, if so my bad.

There is a PR opened that needs reviewing before moving forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci issues related to CI / tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants