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

Where should the option "veratimmodulesyntax" be placed? #211

Closed
chshouyu opened this issue Aug 21, 2023 · 2 comments
Closed

Where should the option "veratimmodulesyntax" be placed? #211

chshouyu opened this issue Aug 21, 2023 · 2 comments

Comments

@chshouyu
Copy link

What about @tsconfig/esm?

We deprecated it in favour of setting #197 to node/bundler.

The @tsconfig/esm contains verbatimModuleSyntax, if @tsconfig/esm is deprecated, do I need to write it in my own tsconfig file?

image

Currently, only these two files contain this option.

@alecmev
Copy link

alecmev commented Aug 29, 2023

I also wonder why moduleResolution was added to it?

@orta
Copy link
Member

orta commented Aug 29, 2023

A reminder that @tsconfig/esm is deprecated.

moduleResolution is what determines how modules are looked up, and because moduleREsolution natively suports node16+ then there's no need for an esm specific runtime

WRT verbatim module syntax, that's your call - I don't necessarily think that needs to be applied to all node projects so it is something you would add if you wanted it - yes

@orta orta closed this as completed Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants