kyranet/veza

[Help Wanted] Rewrite to Strict TypeScript

Closed this issue · 5 comments

kyranet commented

Describe the issue

Veza already supports TypeScript via its typings, but it's support for TS could be greatly improved by rewriting it to strict TypeScript. This might as well fix some hidden bugs.

dshubhadeep commented

@kyranet This seems interesting. I would like to work on this.

  • ❤️1
dshubhadeep commented

@kyranet How do you want me to proceed ?

kyranet commented

Naturally, forking the repository, rewriting it to TS, and PRing it back. Obviously notifying it here (so people can know you're working on it). In case of doubt, you can ask me anytime 👍

  • 👍1
dshubhadeep commented

@kyranet Hey, I have submitted a pull request.
I have converted the js files, but there were some errors that I wasn't able to resolve, such as Object is possibly null and some type errors.
Let me know if there are any additional changes.

  • 🚀1
kyranet commented

Fixed in #15