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

[Question]: TypeScript types? #14

Open
vhesener opened this issue May 6, 2020 · 5 comments
Open

[Question]: TypeScript types? #14

vhesener opened this issue May 6, 2020 · 5 comments

Comments

@vhesener
Copy link

vhesener commented May 6, 2020

Feel free to remove this issue, but I wasn't sure where to ask questions. I noticed there are no types in DefinitelyTyped. Have you heard of anyone that has created TS types for this API already? As a truly lazy dev, I'd like to reuse and recycle :)

@thisissami
Copy link

thisissami commented May 7, 2020

It's your lucky day! I just submitted a PR to DefinitelyTyped with type definitions for this library. DefinitelyTyped/DefinitelyTyped#44534

@vhesener
Copy link
Author

vhesener commented May 7, 2020

Awesome! Yea I couldn't help myself and was playing around with this myself. I came up with basically the same thing, minus the tests. Thanks for the quick turnaround!

@thisissami
Copy link

@ivantsov - the team at DefinitelyTyped (the typescript definitions library) have a question about your intent with the current definition file in your repo (since it hasn't been published in NPM). Could you answer them? DefinitelyTyped/DefinitelyTyped#44534 (comment)

@ivantsov
Copy link
Owner

Hey @thisissami, TS file was added by this PR #6. It was a contribution from one of the users. I don't have any plans for adding TS support for this package, but would check a PR.

@EdmundasR
Copy link

EdmundasR commented Dec 7, 2021

@ivantsov the typing file is not present once installed - you can go to node_modules of redux-subscriber and the index.d.ts is not present neither in root or lib dirs.

edited: link to publishing npm module with type: https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html

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

No branches or pull requests

4 participants