-
Notifications
You must be signed in to change notification settings - Fork 36
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
Typings #80
Comments
I created type definitions: https://github.com/DanielRose/DefinitelyTyped/tree/stompit If no-one complains, I'll send a PR to DefinitelyTyped. |
@DanielRose how about submitting a PR here too? I think a better place for TS typings is the original repo. |
@gdaws It is up to the maintainer what would be preferred. The type definitions could be bundled with the package as well. |
The type definitions were merged and can now be found in https://www.npmjs.com/package/@types/stompit |
Hi, could be that the merged types are now obsolete? For example, in the type Thanks! |
@hazzo The type
Their types define a host property (among others):
And so on. However, an IPC connection does not have a host (since it always is the same system). So to see the correct options, you'll probably have to cast/declare your options objects with the correct type. |
@DanielRose thanks! I missed completely the extended interfaces!!! T_T |
I would love to use this library with Typescript rather than Javascript, are there any typings available?
The text was updated successfully, but these errors were encountered: