-
Notifications
You must be signed in to change notification settings - Fork 25
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
modernization of dependencies #26
Comments
I've found that using DefinitelyTyped manually causes the project to use of older (deprecated) packages. I think it's a good idea to move towards using typings for the TS dependencies, most of which seem to be pulled from DefinitelyTyped. Any objections? |
Sorry for the late response. This project may not be continued by me, because I don't have enough resource to contribute on this. Therefore, I'm willing to hand this project over to other contributors who are interested and would like to maintain. Please leave comments in #28 if interested. Sorry again, and thanks. |
I have no objections to it, I think it's a good idea and necessary for the future of koko. Do you have your fork building fine with those changes or are you still working on it? |
it's building fine but not all features are working @garetmckinley |
Alright. Is it something that you'll be able to tackle yourself or would you rather commit the changes to a branch in this repository so we could tag team some of the issues? |
I need to clean up a lot of the source and then I'll merge it to a dev
|
Then we can evaluate how the work can be split if necessary before merging
|
hey everyone, I'm trying to do some maintenance to facilitate future features/fixes.
I'm working at mmarkman/koko
The text was updated successfully, but these errors were encountered: