You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Removing lines 50 and 54 in EleventyVite.js (in 6bae645) leads to Rollup generating empty JS files again (also see #3) when files like feeds.xml, robots.txt, or sitemap.xml are processed by Eleventy and placed in the Vite public folder in the temp directory, e.g. .11ty-vite/public/.
Generated an empty chunk: "build"
Generated an empty chunk: "feed"
Generated an empty chunk: "robots"
Generated an empty chunk: "sitemap"
Normally, those files should be ignored by Vite because they're in the public folder, yet they are explicitly being added to viteOptions.build.rollupOptions.input with all other items of the results array on eleventy.after. Adding the items of the results array to rollupOptions.input is needed so that all HTML files are added as entry points. So re-adding the filter() makes the most sense IMO.
Could create a PR for this – just LMK @zachleat! 🤗
The text was updated successfully, but these errors were encountered:
matthiasott
changed the title
Empty chunks generated by Rollup for assets in Vite’s publicfolder
Empty chunks generated by Rollup for assets in Vite’s public folder
Jul 18, 2022
Removing lines 50 and 54 in EleventyVite.js (in 6bae645) leads to Rollup generating empty JS files again (also see #3) when files like
feeds.xml
,robots.txt
, orsitemap.xml
are processed by Eleventy and placed in the Vitepublic
folder in the temp directory, e.g..11ty-vite/public/
.Normally, those files should be ignored by Vite because they're in the
public
folder, yet they are explicitly being added toviteOptions.build.rollupOptions.input
with all other items of theresults
array oneleventy.after
. Adding the items of theresults
array torollupOptions.input
is needed so that all HTML files are added as entry points. So re-adding the filter() makes the most sense IMO.Could create a PR for this – just LMK @zachleat! 🤗
The text was updated successfully, but these errors were encountered: