-
Notifications
You must be signed in to change notification settings - Fork 42
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
Port from flow to Typescript #52
Conversation
Turns off smoke test msg tests - typing eliminates those
@@ -0,0 +1,23 @@ | |||
declare module "dryadic" { |
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.
dryadic will also be ported to Typescript. Afterwards this declaration file will be removed.
@@ -142,7 +140,7 @@ class Buffer { | |||
/** | |||
* Deallocate the Buffer, freeing memory on the server. | |||
*/ | |||
free(): Promise<number> { |
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.
Old version was incorrectly typed. it returns void
import { MsgType, CallAndResponse } from "../../Types"; | ||
import Server from "../../server/server"; | ||
|
||
interface Properties {} |
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.
What is this Properties? It's not really used here. Purpose is unclear.
…s missing This allows you to install from git source, not just from npm
Typescript esModuleInterop works fine for lodash
8e893a0
to
0931a25
Compare
e516c89
to
7b4914e
Compare
Converts all files to
.ts
and changes the typing from flow annotations to TypeScript.The library is now usable with JavaScript or TypeScript. When using VSCode you get intelligent error checking and auto-completion even when using JS.
The API remains the same. The published JS is the same interface. AFAICT nothing breaks.
Fixes #44