-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
perf: bundle create-vite #9034
perf: bundle create-vite #9034
Conversation
✅ Deploy Preview for vite-docs-main ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -0,0 +1,126 @@ | |||
// @ts-check |
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.
this file needs to be js because it is imported from packages/vite/rollup.plugin.ts
and it won't be transpiled.
✅ Deploy Preview for vite-docs-main ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
This looks good to me. Was hoping git would detect as a move instead so we get to preserve the git history for the index.js
, but I don't think that's a big issue.
@sapphi-red would you resolve the conflicts when you have a chance? I think we could release this in tandem with the next Vite minor (as a new create-vite minor) |
@patak-dev Which should I rebase on, main or #9406? |
@sapphi-red mmm... let's wait for today's meeting, we may have a decision about #9406 then. |
Looks like #9406 is merged, so this can be rebased on |
af67914
to
02caedf
Compare
02caedf
to
c785eec
Compare
c785eec
to
fb49a3b
Compare
I've rebased on main 👍 |
Description
Bundles and minifies create-vite to reduce download time and requests.
Additional context
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).