-
Notifications
You must be signed in to change notification settings - Fork 408
feat(ui): Switch to ESM only builds #7264
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
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/elements
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/themes
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
| import rspack from '@rspack/core'; | ||
| import packageJSON from './package.json' with { type: 'json' }; | ||
| import path from 'path'; | ||
| import { fileURLToPath } from 'url'; | ||
| import { merge } from 'webpack-merge'; | ||
| import ReactRefreshPlugin from '@rspack/plugin-react-refresh'; | ||
| import { svgLoader, typescriptLoaderProd, typescriptLoaderDev } from '../../scripts/rspack-common.js'; | ||
|
|
||
| const __filename = fileURLToPath(import.meta.url); | ||
| const __dirname = path.dirname(__filename); | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nikosdouvlis do we still need this file ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tagging @dstaley to answer both questions at the same time :)
rspack.config.mjs should be rspack.config.js - I think I managed to stage only some of the changes and the delta here is weird. Pushing a fix in a bit
| "clean": "rimraf ./dist", | ||
| "dev": "rspack serve --config rspack.config.js", | ||
| "dev:origin": "rspack serve --config rspack.config.js --env devOrigin=http://localhost:${PORT:-4001}", | ||
| "dev": "rspack serve --config rspack.config.mjs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"type": "module" will treat all files in the current directory as ESM by default, so shouldn't this be rspack.config.js? I don't think we need the .mjs extension.
d1aa413 to
54fbc48
Compare
Description
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change