-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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(worker): support worker format, plugins and rollupOptions (#6191) #6351
Conversation
Duplicate of #6243 |
@egasimus @patak-dev new PR here!👋 |
Thanks @poyoho! |
@patak-dev Sorry if offtopic, but does this mean a new release soon? Can't seem to get Vite from Git working with my setup and I'm really eager to try this out. |
@egasimus this will be part of vite 2.8. We plan to release vite@2.8.0-beta.0 during the next week if everything goes right |
seems not work ? I use coffeescript and vite v2.8.0-beta.3 I forked the vite and write a demo error code in there : |
Hey @eyun-tv! Thanks for testing this feat! Would you create an issue with that demo as a minimal reproduction so we can better track this bug? (if you can, you can upload it in https://vite.new) |
Thanks for the reproduction @eyun-tv, please create a new bug report with it so we can properly track it. |
hi @eyun-tv I think maybe the error in coffee plugin coffee = (config = {}) => {
return {
name: 'coffee',
transform: function(src, id) {
var code, map;
if (/\.coffee$/.test(id)) {
({
js: code,
sourceMap: map
} = CoffeeScript.compile(src, config));
return {code, map};
}
}
};
}; vite will resolve the request id to |
Description
fix: #6191
Additional context
shallow copy of the plugin config
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123
).