Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

Lacking start script #1

Closed
Alacho2 opened this issue Jan 16, 2020 · 7 comments
Closed

Lacking start script #1

Alacho2 opened this issue Jan 16, 2020 · 7 comments

Comments

@Alacho2
Copy link

Alacho2 commented Jan 16, 2020

The ./dist/index.js is lacking in the project

@Jack-Works
Copy link
Owner

Oh you can build the script by tsc or run it directly by ts-node -T src/index.ts

By the way, the matrix-js-sdk should be in the ../matrix-js-sdk/

@Jack-Works
Copy link
Owner

And if you're looking for a simpler but somehow typed API, you can use https://github.com/turt2live/matrix-js-bot-sdk this project.

@Alacho2
Copy link
Author

Alacho2 commented Jan 20, 2020

Sweet, that's very nice. I'm trying to fork your code to function for Flow.JS :)

@Jack-Works
Copy link
Owner

Sweet, that's very nice. I'm trying to fork your code to function for Flow.JS :)

Hi, these days matrix-js-sdk has changed a lot. It coverts all files to the ES6 module and I help them to convert all classes to ES6 class (still in progress). This will make static analysis easier. Many of the tricks in this project is for CommonJS and ES5 class and this is no longer needed.

After all the classes upgraded to ES6, I will review this project and up it to date.

@Jack-Works
Copy link
Owner

On the other way, @huan has provided a TypeScript definition written by hand (matrix-org/matrix-js-sdk#983 (comment)) you maybe want to check it out to create a flow version.

@huan
Copy link

huan commented Mar 3, 2020

I'm very happy to see we have this project and it's in progress!

It will be great that the codebase can move to ES6 (or TypeScript, which will be better) soon, so we can get an accurate version of the Typing definition by automatic generation instead of writing from scratch with lots of bugs like what I did in the past months. :D

@Jack-Works
Copy link
Owner

@Alacho2 @huan I have just re-generate the types for matrix-js-sdk 5.2.0, you may want to check it out

and I added the start script so I'm going to close this issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants