-
Notifications
You must be signed in to change notification settings - Fork 338
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
feat/sandbox #2306
feat/sandbox #2306
Conversation
Must be merged after #2294 with fixed conflicts. And fix-import-hell plugin must be rewritten to ts. |
64babff
to
21108b7
Compare
@aluarius, I'll need your help to QA my changes here. If you have test nuxt app, check out how build works cuz I don't believe myself. I changed mjs build logic, so this likely must work with vitest without additional effort. (see #2254 (comment)). @asvae I guess you would be interested in |
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.
Everything treeshakes fine for me - components, styles.
closes #2273 - nuxt tree-shaking
closes #2280 - rename to sandbox
closes #2307
closes #2255
this pr and #2279 closes #2274 (need to double check)
In addition to sandbox here, I removed CSS import from
.mjs
file, so pure node runners (without CSS loader) like vitest can handle it.I also removed extra
import 'puk.js'
which I assume vite thought has side-effects because ofvendor.js
imported.