siefkenj / unified-latex

Utilities for parsing and manipulating LaTeX ASTs with the Unified.js framework
MIT License
85 stars 20 forks source link

Use ESM build of vite #90

Closed pddg closed 5 months ago

pddg commented 5 months ago

npm test shows following warning message.

The CJS build of Vite's Node API is deprecated. See https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.

It seems that package.json in root does not have "type": "module" and vite.config.ts is used. I did following:

siefkenj commented 5 months ago

Thanks for this! I think the cleaner solution is to add type: "module" to the root package.json. Could you modify this PR to do that instead?

pddg commented 5 months ago

Sure. Unnecessary commits were removed and force pushed.

pddg commented 5 months ago

Edited comments for clarity of changes.

siefkenj commented 5 months ago

Thanks for the fix!