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

(feature) Added types with docs for the all package #19

Closed
wants to merge 1 commit into from
Closed

(feature) Added types with docs for the all package #19

wants to merge 1 commit into from

Conversation

ggallovalle
Copy link

Hey, I added types for the package. Hope you can merge it and publish a new version 😄

I'll be adding types to other hapipal projects, which one do you think I should do next?

Also I'm thinking of bringing forward the hapipal ecosystem to work with Typescript

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9411ca4 on ggallovalle:master into 55c7c8f on hapipal:master.

@devinivy
Copy link
Member

Hey @ggallovalle thanks for the contribution— glad to see you are interested in moving our Typescript support forward! Given the plan described in hapipal/discuss#4 we were actually planning to place all types for this ecosystem in a single repository, sort of like a mini @types project that we control ourselves. So I am going to close this, but this is still a useful contribution! It will likely just live in a different repository. Look for a comment from me shortly in that other issue 👍

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

Successfully merging this pull request may close these issues.

3 participants