-
Notifications
You must be signed in to change notification settings - Fork 471
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
Upgrade to ESM build of Vite #1985
Comments
Hello @yurishkuro should we run the |
why? |
hey @yurishkuro... in order to get rid of the warning we need to rename the
however, on doing this it throws an error on line 60 in
i fixed this by reffering to https://www.npmjs.com/package/rollup-plugin-visualizer. The warning is fixed now :). Should i open a PR? |
This commit fixes the CJS deprication warning. Signed-off-by: shivam <[email protected]>
This commit fixes the CJS deprication warning Signed-off-by: shivam <[email protected]>
This commit fixes the CJS deprication warning Signed-off-by: shivam <[email protected]>
The previous commit changed the config from CJS to ESM which caused 2 tests to fail. This commit fixes the above issue by replacing 'require' with 'import' throughout the project Signed-off-by: shivam <[email protected]>
This commit fixes the CJS deprication warning by renaming vite.config.ts to vite.config.mts. Signed-off-by: shivam <[email protected]>
When running
yarn start
the following warning is printed:According to the wiki, the CJS build will be removed in Vite v6.
The text was updated successfully, but these errors were encountered: