Skip to content

Update VS code to 1.49.3 #2081

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

Merged
merged 2 commits into from
Oct 7, 2020
Merged

Update VS code to 1.49.3 #2081

merged 2 commits into from
Oct 7, 2020

Conversation

code-asher
Copy link
Member

@code-asher code-asher commented Sep 10, 2020

I didn't have to make any changes except add a new file to the coder.js build script.

Might fix #1015 but I don't have any means of testing it.

@code-asher code-asher requested a review from nhooyr as a code owner September 10, 2020 22:46
@0x4007
Copy link

0x4007 commented Sep 23, 2020

I cloned and tried to build -- do you know what I might be missing?

yarn build v0.24.6
(node:13301) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
$ ./ci/build/build-code-server.sh
src/node/app/vscode.ts:15:8 - error TS2307: Cannot find module '../../../lib/vscode/src/vs/server/ipc' or its corresponding type declarations.

15 } from "../../../lib/vscode/src/vs/server/ipc"
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:6:32 - error TS2307: Cannot find module '../../lib/vscode/src/vs/server/ipc' or its corresponding type declarations.

6 import { Args as VsArgs } from "../../lib/vscode/src/vs/server/ipc"
                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:123:3 - error TS2322: Type '{ auth: { type: typeof AuthType; description: string; }; password: { type: "string"; description: string; }; cert: { type: typeof OptionalString; path: true; description: string; }; "cert-key": { ...; }; ... 27 more ...; verbose: { ...; }; }' is not assignable to type 'Options<Required<Args>>'.
  Object literal may only specify known properties, and '"user-data-dir"' does not exist in type 'Options<Required<Args>>'.

123   "user-data-dir": { type: "string", path: true, description: "Path to the user data directory." },
      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:292:12 - error TS2339: Property 'verbose' does not exist on type 'Args'.

292   if (args.verbose) {
               ~~~~~~~

src/node/cli.ts:309:12 - error TS2339: Property 'verbose' does not exist on type 'Args'.

309       args.verbose = true
               ~~~~~~~

src/node/cli.ts:313:12 - error TS2339: Property 'verbose' does not exist on type 'Args'.

313       args.verbose = false
               ~~~~~~~

src/node/cli.ts:317:12 - error TS2339: Property 'verbose' does not exist on type 'Args'.

317       args.verbose = false
               ~~~~~~~

src/node/cli.ts:321:12 - error TS2339: Property 'verbose' does not exist on type 'Args'.

321       args.verbose = false
               ~~~~~~~

src/node/cli.ts:325:12 - error TS2339: Property 'verbose' does not exist on type 'Args'.

325       args.verbose = false
               ~~~~~~~

src/node/cli.ts:335:8 - error TS7053: Element implicitly has an 'any' type because expression of type '"user-data-dir"' can't be used to index type 'Args'.
  Property 'user-data-dir' does not exist on type 'Args'.

335   if (!args["user-data-dir"]) {
           ~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:337:5 - error TS7053: Element implicitly has an 'any' type because expression of type '"user-data-dir"' can't be used to index type 'Args'.
  Property 'user-data-dir' does not exist on type 'Args'.

337     args["user-data-dir"] = paths.data
        ~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:340:8 - error TS7053: Element implicitly has an 'any' type because expression of type '"extensions-dir"' can't be used to index type 'Args'.
  Property 'extensions-dir' does not exist on type 'Args'.

340   if (!args["extensions-dir"]) {
           ~~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:341:5 - error TS7053: Element implicitly has an 'any' type because expression of type '"extensions-dir"' can't be used to index type 'Args'.
  Property 'extensions-dir' does not exist on type 'Args'.

341     args["extensions-dir"] = path.join(args["user-data-dir"], "extensions")
        ~~~~~~~~~~~~~~~~~~~~~~

src/node/cli.ts:341:40 - error TS7053: Element implicitly has an 'any' type because expression of type '"user-data-dir"' can't be used to index type 'Args'.
  Property 'user-data-dir' does not exist on type 'Args'.

341     args["extensions-dir"] = path.join(args["user-data-dir"], "extensions")
                                           ~~~~~~~~~~~~~~~~~~~~~

src/node/entry.ts:6:49 - error TS2307: Cannot find module '../../lib/vscode/src/vs/server/ipc' or its corresponding type declarations.

6 import { CliMessage, OpenCommandPipeArgs } from "../../lib/vscode/src/vs/server/ipc"
                                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/node/entry.ts:45:48 - error TS7053: Element implicitly has an 'any' type because expression of type '"user-data-dir"' can't be used to index type 'Args'.
  Property 'user-data-dir' does not exist on type 'Args'.

45   logger.info(`Using user-data-dir ${humanPath(args["user-data-dir"])}`)
                                                  ~~~~~~~~~~~~~~~~~~~~~

src/node/entry.ts:47:50 - error TS7053: Element implicitly has an 'any' type because expression of type '"extensions-dir"' can't be used to index type 'Args'.
  Property 'extensions-dir' does not exist on type 'Args'.

47   logger.trace(`Using extensions-dir ${humanPath(args["extensions-dir"])}`)
                                                    ~~~~~~~~~~~~~~~~~~~~~~


Found 17 errors.

error Command failed with exit code 2.
nv:code-server Pavlovcik$

@code-asher
Copy link
Member Author

code-asher commented Sep 28, 2020

It looks like the patch wasn't applied. Check out https://github.com/cdr/code-server/blob/v3.5.0/doc/CONTRIBUTING.md#build. In particular the yarn vscode step is what applies the patch.

@code-asher code-asher linked an issue Sep 30, 2020 that may be closed by this pull request
@DAddYE
Copy link

DAddYE commented Oct 2, 2020

Hi, I got the same error and was after yarn vscode. Any idea what could be?

@DAddYE
Copy link

DAddYE commented Oct 2, 2020

Hi, I got the same error and was after yarn vscode. Any idea what could be?

Oh, figure out, basically I did yarn vscode:diff, which wiped the patch :)

Works well, and I confirm fixed the issue on iPad.

@code-asher
Copy link
Member Author

Thank you for testing it!!

@code-asher code-asher changed the title Update VS code to 1.49.0 Update VS code to 1.49.3 Oct 5, 2020
Copy link
Contributor

@nhooyr nhooyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@code-asher code-asher merged commit 402f5eb into master Oct 7, 2020
@code-asher code-asher deleted the vscode-1.49.0 branch October 7, 2020 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

iOS copy and cut keyboard shortcuts are not working
4 participants