-
Notifications
You must be signed in to change notification settings - Fork 4
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
Remove any types and implement proper type assertions #16
Conversation
Will do! |
@thepassle I'm not going to lie. There needs some serious work done to make typing to work properly in this project. |
@thepassle It would be really great if you could tag this PR with |
Sorry for the late reply, I've had some IRL stuff to attend to this past week, and I've also been considering the approach of this project, and potentially just moving it over to JS with JSDoc, but I'm still unsure on what to do. Does adding a |
Thank you @thepassle ! The PR started counting as soon as you added the label. I wish you best of luck with your project! |
In this PR I have started to remove any-types by creating proper type-asssertions.
I have also moved some conditionals to make it both easier to read and for typescript to easier infer the correct type.
It is not a complete fix, but it should be a good start.
Before merging with master I would try to pull locally and test that it doesn't break anything.
I couldn't figure out how to use the correct ts-lint and how to run it. But I am fairly certain that it should be all good
#14