-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
Support for ESM yet? #13817
Comments
The nest.js team has been resistant to change for too long. I am sure that soon we will see a new framework that implements modern development principles, including esm-decorators. |
There's no way to support ESM decorators right now due to missing features like parameters decorators.
I don't think so. Moving to ESM is not an easy task an you can use ESM-only packages with nestjs apps just fine. |
Funny enough, 99,9% of people asking for ESM support don't really need it (this feature really makes 0 difference for most Nest apps out there). And yes, using dynamic imports for ESM-only packages might be annoying and I get that, but believe me or not, everyone would be struggling much more if we switched to ESM by default right now. No need to create yet another issue to track this. Once it's safe to migrate, we'll do this |
Originally posted by @kamilmysliwiec in #7021 (comment)
The text was updated successfully, but these errors were encountered: