-
-
Notifications
You must be signed in to change notification settings - Fork 171
add experimental-strip-types in app-ts-esm #798
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
base: main
Are you sure you want to change the base?
Conversation
|
Once strip types ships in Node v20, I'd recommend shipping a new version of this as a semver-major. |
|
Why is |
I tried to run it without this flag, but I got an error. I don't know why |
|
fastify/fastify-autoload#442 < I think fastify-autoload would need some fixing. I think we can access process.features to detect if there is the support instead of relying on the version number. |
Now that Node 20 is in maintenance mode and experimental-strip-types is the default on LTS 22 and 24, should I still leave --experimental-strip-types? |
|
No, drop node 20 and let's move in the future (I'll ship a major). |
I modified the app-ts-esm template by inserting the indicated tsconfig
https://nodejs.org/en/learn/typescript/run-natively#running-typescript-code-with-nodejs and erasableSyntaxOnly.
This way the dev script can directly launch the ts file with --experimental-strip-types
Checklist
npm run testandnpm run benchmarkand the Code of conduct