microsoft/node-jsonc-parser

export mjs for esm

xiaoxiangmoe opened this issue · 5 comments

https://nodejs.org/api/esm.html#esm_conditional_exports

Add support for esm. Compile file into ./lib/esm/main.mjs.

Currently ts does not support mjs, can we consider using rollup packaging?

see #37 and #38. Choose one to merge and close this issue.

Sorry, I have no experience with the new, experimental exports property but I'm afraid the suggested change will break current users.

But maybe you can elaborate a bit what the advantages are. What packagers/ runtime do already support it?

Hi @xiaoxiangmoe, see my comment above. I'm still waiting for some explanation what the advantages of the change is.

We already support the module property. We don't have the proper esm inport syntax (no file extension), but from what I know, rollup and webpack can work with it.

Closing. I need more help here to understand what the advantages are.