✨ Coder.com for PR #4171 deployed! It will be updated on every commit.

• Host: https://codercom-g5oimt3mj-codercom.vercel.app/docs/code-server
• Last deploy status: success
• Commit: bdca804
• Workflow status: https://github.com/cdr/code-server/actions/runs/1254636225

error Your lockfile needs to be updated, but yarn was run with --frozen-lockfile.

Do we need to update the yarn.lock file?

Codecov Report

Merging #4171 (bdca804) into main (4f3c8a5) will increase coverage by 0.94%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             main    #4171      +/-   ##
==========================================
+ Coverage   64.22%   65.17%   +0.94%     
==========================================
  Files          36       37       +1     
  Lines        1873     1898      +25     
  Branches      379      382       +3     
==========================================
+ Hits         1203     1237      +34     
+ Misses        569      562       -7     
+ Partials      101       99       -2     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f3c8a5...bdca804. Read the comment docs.

This PR likely needs to be rebased and updated with the latest vscode commit

I wasn't sure what you meant by "updated with the latest vscode commit":

Option 1

1. go to https://github.com/cdr/vscode
2. hit Fetch upstream
3. grab the latest vscode commit
4. use that in vendor/package.json

Option 2

1. go to https://github.com/cdr/vscode
2. grab the latest vscode commit: affcb6753278c014d1eea83664e6fbf5d702852a
3. use that in vendor/package.json

Option 3.

1. look at main branch in code-server
2. use vscode commit in vendor/package.json

I rebased and went with Option 3: 9cb5fb3759f46b10bc66e676fa7f44c51e84824b

I hope that's right. I'll wait for you to check before we merge this.

Tentatively resolved in #4010