Skip to content

Rewrite code into typescript for easier integration with other npm projects#27

Open
therealadityashankar wants to merge 3 commits intopainebenjamin:mainfrom
therealadityashankar:main
Open

Rewrite code into typescript for easier integration with other npm projects#27
therealadityashankar wants to merge 3 commits intopainebenjamin:mainfrom
therealadityashankar:main

Conversation

@therealadityashankar
Copy link
Copy Markdown
Contributor

@therealadityashankar therealadityashankar commented Sep 19, 2025

closes #7

@therealadityashankar therealadityashankar changed the title Rewrite code into typescript for easier integration with other npm libraries Rewrite code into typescript for easier integration with other npm projects Sep 19, 2025
@therealadityashankar
Copy link
Copy Markdown
Contributor Author

also, @painebenjamin if you can consider #23 I would be a little more motivated, currently when I search for "wake word libraries" on https://www.npmjs.com/search?q=wake%20word many commercial solutions from davoice show up, and I would be more motivated to contribute if this package gets many more users

I believe a name, more resonating with open source projects, such as "simple wake word" or "oss wake word" or something else, would likely get much more traction :)

@therealadityashankar therealadityashankar marked this pull request as ready for review October 12, 2025 10:25
@therealadityashankar
Copy link
Copy Markdown
Contributor Author

yay okie fixed it into typescript @painebenjamin , if you want to review!, I am using parceljs in the typescript code (as I find it easier!)

@painebenjamin
Copy link
Copy Markdown
Owner

Wow, you did a lot! Give me a little time to review and test and I'll merge it in. Thank you!

@therealadityashankar
Copy link
Copy Markdown
Contributor Author

oh just fyi you run everything with parcel now, instead (because it was easy for me 😅 ), you might have to modify a bit as well

@therealadityashankar
Copy link
Copy Markdown
Contributor Author

@painebenjamin any updates here?

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.

Add Typescript support

2 participants