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

Add TypeScript type definitions #34

Merged
merged 1 commit into from
Aug 28, 2017
Merged

Add TypeScript type definitions #34

merged 1 commit into from
Aug 28, 2017

Conversation

druska
Copy link
Contributor

@druska druska commented Aug 24, 2017

Thanks for the awesome queueing framework! I've added TypeScript type definitions. If you don't want to be responsible for maintaining this, we can add them to DefinitelyTyped instead. Integrated is preferable for TypeScript users.

@coveralls
Copy link

coveralls commented Aug 24, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 0d388af on druska:master into e602c0f on timgit:master.

@timgit
Copy link
Owner

timgit commented Aug 25, 2017

@druska, thanks for the PR. I've yet to use this in a Typescript project yet, but I can see how handy this would be. Did you hand-create this, or did you use a tool to auto-gen most of it? If I were to take over maintenance, I'd like to automate it as much as possible. That is, something I could use outside of a full-blown rewrite in TS. 😱

@timgit timgit merged commit af2cc1f into timgit:master Aug 28, 2017
@druska
Copy link
Contributor Author

druska commented Sep 5, 2017

@timgit if the project is written in plain JavaScript, it's something you have to maintain yourself unfortunately.

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

Successfully merging this pull request may close these issues.

3 participants