-
Notifications
You must be signed in to change notification settings - Fork 38
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
Migrating to Typescript #24
Comments
What are the benefits of migrating to TS in your opinion? Basically I'm concerned about the absence of native support in node, so code will need compiling.
|
With Typescript the package from npm would come with typings (*.d.ts files) without having to maintain them separately. Moreover you won't really need to type using jsdoc if you use Typescript and the TS compiler will ensure the source code is free from syntax errors. While I do understand the cons that you described, I have only experienced point 3 out of all of them in my time of using TS for development. |
I am still not convinced, but I have to admit I've never managed a TS project before. |
Hi, I understand some of the concerns you mention @atoppi . Adding an extra step in the build chain can be adding another place of failure, another source of friction for initial onboarding, etc. I'm not sure why you think it is more difficult debugging. Perhaps one main benefit is the addition of types to help narrow down potential bugs. Personally, I moved to TS years ago since I find the advantages outweigh the disadvantages. Newer runtimes like bun and deno can also ingest TS directly. I'd also prefer janode in TS. |
I am not referring to static analysis. Maybe I am worrying too much and that is a false problem, I don't know, as I said I don't have experience in managing TS projects. Anyway I have understood that users really care about type definitions and I will definitely tackle this sooner or later, maybe by working on #5 or migrating the whole project to TS. |
Any plan or interest in migrating the codebase to Typescript? I think it would be more manageable in the long run compared to managing separate d.ts files.
The text was updated successfully, but these errors were encountered: