Skip to content
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

chore: update deltachat-jsonrpc JS dependencies #4989

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

link2xt
Copy link
Collaborator

@link2xt link2xt commented Nov 13, 2023

No description provided.

@Simon-Laux
Copy link
Member

Simon-Laux commented Nov 15, 2023

I get these typescript errors when running npm run build locally:

> @deltachat/[email protected] build:tsc
> tsc

node_modules/@types/node/globals.d.ts:72:13 - error TS2403: Subsequent variable declarations must have the same type.  Variable 'AbortSignal' must be of type '{ new (): AbortSignal; prototype: AbortSignal; abort(reason?: any): AbortSignal; timeout(milliseconds: number): AbortSignal; }', but here has type '{ new (): AbortSignal; prototype: AbortSignal; }'.

72 declare var AbortSignal: {
               ~~~~~~~~~~~

  node_modules/typescript/lib/lib.dom.d.ts:2335:13
    2335 declare var AbortSignal: {
                     ~~~~~~~~~~~
    'AbortSignal' was also declared here.

node_modules/@types/ws/index.d.ts:321:18 - error TS2315: Type 'Server' is not generic.

321         server?: HTTPServer<V> | HTTPSServer<V> | undefined;
                     ~~~~~~~~~~~~~

node_modules/@types/ws/index.d.ts:321:34 - error TS2315: Type 'Server' is not generic.

321         server?: HTTPServer<V> | HTTPSServer<V> | undefined;
                                     ~~~~~~~~~~~~~~


Found 3 errors in 2 files.

Errors  Files
     1  node_modules/@types/node/globals.d.ts:72
     2  node_modules/@types/ws/index.d.ts:321
ERROR: "build:tsc" exited with 2.

maybe we should not update typescript? or depend on a specific version of @types/node?

@link2xt
Copy link
Collaborator Author

link2xt commented Nov 16, 2023

@Simon-Laux

Did you run npm i before running npm run build? For me npm run build executes fine.

maybe we should not update typescript?

Why?

or depend on a specific version of @types/node?

Which version, what is the problem with the version @types/ws depends on? For me it is in the output npm ls -a:

├─┬ @types/[email protected]
│ └─┬ @types/[email protected]
│   └── [email protected]

@link2xt link2xt added the node Nodejs bindings label Jan 6, 2024
@link2xt link2xt force-pushed the main branch 2 times, most recently from 1abb12e to 2af9ff1 Compare March 4, 2024 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
node Nodejs bindings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants