-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
code-server: 4.12.0 -> 4.13.0 #233493
code-server: 4.12.0 -> 4.13.0 #233493
Conversation
7b233c5
to
48b9939
Compare
7f86bd1
to
1005d5a
Compare
I don't think it's good, it's not building |
@@ -1,12 +1,12 @@ | |||
{ lib, stdenv, fetchFromGitHub, buildGoModule, makeWrapper | |||
, cacert, moreutils, jq, git, rsync, pkg-config, yarn, python3 | |||
, esbuild, nodejs_16, node-gyp, libsecret, xorg, ripgrep | |||
, esbuild, nodejs_18, node-gyp, libsecret, xorg, ripgrep |
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.
why did you change to nodejs 18, code-server officially still only supports 16
https://github.com/coder/code-server/blob/2109d1cf6abdf17e620a926239d9e96a07face4d/.github/workflows/release.yaml#LL38C13-L38C13
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.
nodejs_16
has been deprecated, so I'm just checking if this is working with NodeJS 18. I already made the change in openvscode-server at #232887
1005d5a
to
7dcacb0
Compare
@GrahamcOfBorg build code-server |
I guess I can close this PR now, it just doesn't work with NodeJS 18. |
made an issue at code-server to see if they can look into migrating first |
Work is continued in #234289 |
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)