-
Notifications
You must be signed in to change notification settings - Fork 6k
[Feat]: Build with Node 16 #4894
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
Comments
Probably! And then I think we need to update the https://github.com/coder/code-server/blob/main/package.json#L121 And update this: https://github.com/coder/code-server/blob/main/.node-version And then we'll need to update the docs if they mention v14 |
When technically we cant use node16, but the release can run node16, as far i can tell it work pretty well. |
@ghalibansari not sure yet since we have other priorities. You'll either see an update here or see this moved to an upcoming milestone. Hopefully soon though! side note: jawnsy is no longer helping out with the project so please don't tag them. |
@jsjoeio understood, thanks.
|
@ghalibansari same here! If you are using typescript/javascript and seeing errors with code-server 4.2.0 (#5019), try using code-server 4.1.0 until this is resolved:
If you're following the instructions on coder.com to install on Termux then you'll want to install code-server this way:
|
Does it work well for you? In my case it helped with the installation, while breaking extensions, making code-server cool but not so useful... Am I the only one facing this issue? |
yes if you face any issue use, |
The issue persists anyway. I can install code-server but not extensions.
I can assure you, there is no folder in the directory whatsoever. Maybe it is not related to the node version, but it seems like it. I've also tried other ways to install extensions, but without success. |
Indeed, on Termux directly, code-server is trying to delete a folder that it failed to create earlier 🤷♂️. Extensions didn't work for me installing directly in Termux without PRoot. Best way I've found so far is instructions on Coder (with PRoot) and Debian: https://coder.com/docs/code-server/latest/termux Make sure to install Node v14 (I realize the irony of Node v14 in this thread but you can install multiple versions of Node... v14 for code-server). I am using code-server 4.1.0 as there are issues for me with javascript/typescript and v4.2.0. All extensions working perfectly for me: |
This was done in https://github.com/coder/code-server/milestone/39 |
What is your suggestion?
Upstream VSCode now appears to be building with Node.js version 16. We should consider doing the same when we update our fork to the latest code.
https://github.com/microsoft/vscode/blob/39cc1e1c42b2e53e83b1846c2857ca194848cc1d/.github/workflows/ci.yml#L24-L26
Why do you want this feature?
Node v14 is now a maintenance LTS release, and will be EOL 2023-04-30. Switching to v16 gives us an additional year of support, since Node v16 LTS has an EOL of 2024-04-30.
Are there any workarounds to get this functionality today?
Users can build with Node v16 manually, I suppose.
Are you interested in submitting a PR for this?
Sure! I'm not sure what needs to be updated, though, aside from CI. Do we need to update the fork (https://github.com/coder/vscode/) as well?
The text was updated successfully, but these errors were encountered: