search

Subwaytime / vite-aliases

Alias auto generation for Vite
https://www.npmjs.com/package/vite-aliases
MIT License
210 stars 12 forks source link

Update tsup.config so it would compile cjs and modify package.json #44

Closed tsanyqudsi closed 1 year ago

tsanyqudsi commented 1 year ago

fix #43

Subwaytime commented 1 year ago

Have you tested if this breaks the "type": "module" compatability? I have removed this on purpose to move towards ESM, as vite 3 is doing the same. CJS is becoming more outdated

tsanyqudsi commented 1 year ago

Have you tested if this breaks the "type": "module" compatability? I have removed this on purpose to move towards ESM, as vite 3 is doing the same. CJS is becoming more outdated

I just merely add the CJS compatibility ( which doesn't really work apparently ), And you're right. CJS is more and more getting outdated. I don't think this is worth to merge.

But I suggest that we add "migration from < 0.95 to > 0.95". It took me an hour or so just to realize that I have to put "type: module" on my package.json.