Skip to content
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

migrate towards ESM modules #11897

Closed
1 task done
yuntian001 opened this issue Jun 25, 2023 · 3 comments
Closed
1 task done

migrate towards ESM modules #11897

yuntian001 opened this issue Jun 25, 2023 · 3 comments
Labels
needs triage This issue has not been looked into type: enhancement 🐺

Comments

@yuntian001
Copy link

Is there an existing issue that is already proposing this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe it

I hope nest will migrate to ESM modules and would like to ask about the specific migration planning。

Describe the solution you'd like

The default example uses esm

Teachability, documentation, adoption, migration strategy

The default example uses esm

What is the motivation / use case for changing the behavior?

When I write the test file, await import(pathToFileURL(this.dbConfig).href) is not supported. I want to be able to migrate ESM modules. To get top-level support for import(),And ESM is backward compatible with commonjs, which is very friendly.

@yuntian001 yuntian001 added needs triage This issue has not been looked into type: enhancement 🐺 labels Jun 25, 2023
@jmcdo29
Copy link
Member

jmcdo29 commented Jun 25, 2023

#8736

@yuntian001
Copy link
Author

第8736章

It has not responded for two years. Is there any official plan

@jmcdo29
Copy link
Member

jmcdo29 commented Jun 25, 2023

Kamil agreed with the owners of the fastify repo, it's still premature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage This issue has not been looked into type: enhancement 🐺
Projects
None yet
Development

No branches or pull requests

2 participants