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

Update typescript definitions #193

Open
deivi98 opened this issue Aug 22, 2023 · 4 comments
Open

Update typescript definitions #193

deivi98 opened this issue Aug 22, 2023 · 4 comments

Comments

@deivi98
Copy link

deivi98 commented Aug 22, 2023

Typescript definitions are a bit outdated. Can them be updated please?

#120

@deivi98 deivi98 changed the title @jpike88 https://www.npmjs.com/package/@types/node-pushnotifications Update typescript definitions Aug 22, 2023
@miqmago
Copy link
Collaborator

miqmago commented Nov 7, 2023

I've created a new branch that's completely rewritten into typescript.
It's a big refactoring and needs some strong review and test, not only myself.
I think it can be the next big release for this repo and make it easier to maintain.

@alex-friedl since you've been contributing a lot to this repo, I've sent you a message, maybe we can do some thinking together.

@miqmago
Copy link
Collaborator

miqmago commented Feb 2, 2024

@vitalyster would be awesome if you could explain little bit your feedback, did you take a look at the branch? Is it crazy? Please let me know how would you like to collaborate!

@vitalyster
Copy link
Contributor

@miqmago I'm pretty sure it is much easier and less error prone to maintain existing type definitions but not to do a complete rewrite which will add zero value but require a lot of work

@miqmago
Copy link
Collaborator

miqmago commented Feb 6, 2024

@vitalyster thank for your comments. Completely agree on that it is easier to keep code as it is right now. On the other side, in my opinion typescript is becoming the new standard. Also in fact typescript is something that does not require any rewrite, one could change the name of the files and bum! they are in typescript ☺️

The big effort is on the testing area, which is something that anyway needs someone to take a look every once and then.

On the other side, right now the control of type definitions is over this repo which is not really desirable...

Let's see if there is more time available, I'm planning to use this module for long term and it is interesting to do things that are maintainable in a long term vision. The only drawback is that it will not be immediate. Maybe we will have to survive with outdated type definitions except if there is someone who maintains them...

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

3 participants