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

Typescript definitions #42

Closed
kerolloz opened this issue Jun 21, 2021 · 6 comments · Fixed by #61
Closed

Typescript definitions #42

kerolloz opened this issue Jun 21, 2021 · 6 comments · Fixed by #61
Assignees
Labels
enhancement New feature or request

Comments

@kerolloz
Copy link

No description provided.

@juaesto
Copy link

juaesto commented Sep 29, 2022

Are there available Typescript definitions?

@pieper
Copy link
Contributor

pieper commented Sep 29, 2022

Not that I know of. If someone feels strongly they could make a PR.

@pieper pieper closed this as not planned Won't fix, can't repro, duplicate, stale Sep 29, 2022
@hackermd
Copy link
Member

We have type definitions in Slim and it would be nicer to include them in the package distribution.

@pieper
Copy link
Contributor

pieper commented Sep 29, 2022

Thank you @hackermd. Yes, it would make more sense for those to be here along with some tests to ensure they stay up to date if/when the API evolves.

@pieper pieper reopened this Sep 29, 2022
@hackermd
Copy link
Member

@pieper could you help porting them over from the application into the library?

For the tests, we may need to introduce typescript as a dev dependency.

@pieper
Copy link
Contributor

pieper commented Sep 29, 2022

I don't have any particular need for typescript but perhaps @juaesto or @kerolloz would be willing to help out.

@hackermd hackermd added the enhancement New feature or request label Dec 12, 2022
@hackermd hackermd linked a pull request Dec 12, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants