-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
suggestion: convert the codebase to typescript #274
Comments
@hichemfantar I think I'm going to just venture down this path. Our process needs some extensions/customizations and I need to move forward, see #284 I may just do this, but I can't commit to maintaining it. Preferably we could commit here, but if not, we can do so elsewhere (and hopefully in the end, merge back here) |
I can def migrate the project to ts but like you said, there doesn't seem to be much movement so i'm worried it might be in vain and i also have a pr that was approved but not merged for some reason #279. |
@hichemfantar I'm just going to move forward with it. I need it, hopefully it comes back home here. I'll take a shot at merging your PR into mine before I start changing everything. |
I'm moving into my userland domain with generation, and my modifications have been working well, especially the new Nonetheless, I hope my PR finds a home here for community involvement. I don't want to try and build a separate community if we don't need to. |
I encourage you to publish a fork and not wait, you can always merge back into this repo when the maintainers approve |
the advantages of typescript need no introduction so i'm asking if the team is open to convert this repo to typescript in accordance with other pmndrs repos.
i can work on this.
The text was updated successfully, but these errors were encountered: