-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
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
).