-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[Feat]: Upgrade vscode to 1.70 #5417
Comments
So some first steps (#5422) towards the 1.70 look fine. Jupyter is working, |
@benz0li feel free to have a look on the latest CI artifacts if you can detect some odd bugs. (https://github.com/coder/code-server/actions/runs/2815650333) 😉 |
Will do. |
I have deployed the latest latest CI artifact for |
Confirmed.
|
JupyterLab Terminal: ╰─ code-server --version
4.5.1 c5c16630adb086bf2cfed1ba6d678ef2ac9d06e1 with Code 1.70.1 ╰─ echo $PATH
/home/benz0li/.local/bin:/opt/code-server/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin ╰─ which code-server
/opt/code-server/bin/code-server code-server Terminal: ╰─ code-server --version
1.70.1
2062a59ca1a586d8a6e7bf483841085a94c440a4
x64 ╰─ echo $PATH
/opt/code-server/lib/vscode/bin/remote-cli:/home/benz0li/.local/bin:/opt/code-server/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin ╰─ which code-server
/opt/code-server/lib/vscode/bin/remote-cli/code-server I can live with this 😉. Related to #5335 (comment) ff. I might do some further testing tonight. |
@benz0li @jsjoeio @code-asher can we publish this build as beta and test the obviously resolved bugs like #5335 and close it? |
ℹ️ I am using # set PATH so it includes user's private bin if it exists
if [ -d "$HOME/bin" -a "$SHLVL" = 1 -a ! "$TERM_PROGRAM" = "vscode" ] ; then
PATH="$HOME/bin:$PATH"
fi
# set PATH so it includes user's private bin if it exists
if [ -d "$HOME/.local/bin" -a "$SHLVL" = 1 -a ! "$TERM_PROGRAM" = "vscode" ] ; then
PATH="$HOME/.local/bin:$PATH"
fi in I haven't figured out a proper solution for VS Code on macOS yet, as PATH is set in multiple places. |
@benz0li great. Thanks for testing. Two features/patches required some more adjustments. One is regarding the language packs, the other is about settings storage. From my point of view both patches/features should work correctly at the moment but I'm not sure. |
I can't say anything about that because I do not work @coder. But if a two-month rhythm is followed, [at least] one release candidate would be a good idea – for in-depth testing before the actual release.
@bala You may test at https://vscode-r.jupyter.b-data.ch with Image |
side note: we had an issue with publishing a beta version to npm. we just fixed that. yarn global add code-server@beta will get you the latest off https://www.npmjs.com/package/code-server/v/4.5.1-beta-3335d0a456da0ea639b1e010aeb9801a876a9323 |
Functionality modified by patches tested and found to work:
|
Menu entry |
@fritterhoff Regarding #5417 (comment): What does |
Since #5360 fixes the script it self and not the way it is called the |
Yours is not the output of the official script. That is why I am asking. |
Whooopss... that was to simple 😁
|
In addition: (base) jovyan@jupyter-ritterho-40hm-2eedu:/usr/lib/code-server/lib/vscode/bin/remote-cli$ ls /usr/lib/code-server/lib/vscode/bin/remote-cli
code.cmd code-darwin.sh code-linux.sh
(base) jovyan@jupyter-ritterho-40hm-2eedu:/usr/lib/code-server/lib/vscode/bin/remote-cli$ There is no code-server script only a code ... I have no idea whats wrong here 🤔 The script it self works fine, so the patch seems to be applied. (base) jovyan@jupyter-ritterho-40hm-2eedu:~$ code-linux.sh --version
1.70.1
2062a59ca1a586d8a6e7bf483841085a94c440a4
x64 |
Maybe #5335 (comment)? Do you still delete the In my case there is a link from |
Ah your right. My Dockerfile contains a |
I wanted to wait until getting the 1.70 update in but in order to get #5442 released I think we should make a release candidate today and then a regular release pretty soon after. Working on that right now. |
Nice find! I opened an issue and will fix it real swift #5448 |
3 WAY MERGE VIA CODE SERVER!!!!!!!!!! |
Completed via #5422 |
I can confirm this is now fixed. P.S.: I have deployed v4.6.0-rc.1 at https://vscode-r.jupyter.b-data.ch, Image |
What is your suggestion?
Upgrade vscode to 1.70
Why do you want this feature?
Are there any workarounds to get this functionality today?
Are you interested in submitting a PR for this?
The text was updated successfully, but these errors were encountered: