-
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
Update Code to 1.89.0 #6783
Update Code to 1.89.0 #6783
Conversation
Functionality [modified by patches] tested and found to work:
Workspaces and Jupyter Notebooks also work fine: Cross reference: |
@code-asher I have built as usual. Somehow JupyterLab Terminal code-server --version
code-server Terminal code-server --version
I have no idea why. Just wanted to let you know. |
How strange! I believe it just reads Yeah I checked and the |
VS Code v1.89.0 is built off commit microsoft/vscode@b58957e. I will update this merge request accordingly. |
Updated on their side, too: |
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.
Awesome, thank you!
@code-asher I would like to propose the following: Release candidate (RC)
👉 The Release
Footnotes |
@code-asher With the "Alternative", the patch releases are in sync. And why not keep the minor releases in sync, too? |
It does sound reasonable to keep them in sync, unfortunately sometimes we make fixes or additions to code-server other than updating VS Code, so as soon as that happens we would be permanently out of sync, I think. |
IMHO updates other than Breaking changes in Other than that, minor and patch releases remain in sync with upstream |
I would not be comfortable releasing a feature update as a patch, I think. But I agree that those updates can just wait until the next release. I am open to experimenting with this, if we do come across a feature update that absolutely has to be released early for some reason then we can revisit. |
I put out v4.89.0-rc.1! |
An alternative would be to use semver builds v4.89.1+coder1 like k3s.io is using (e.g. v1.30.0+k3s1). I think going back to v1.89.1+coder1 is not an option ;) |
Fixes #6782