Skip to content
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

Patch 1 #5

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Patch 1 #5

wants to merge 9 commits into from

Conversation

thestephenmarshall
Copy link

Description 📖

This pull request

Background 📜

This was happening because

The Fix 🔨

By changing

Screenshots 📷

KieranP and others added 9 commits May 10, 2024 17:11
This fixes an issue where upgrading to Vite5 was merging old Vite4 and Vite5 manifests together, creating JS errors.
This was broken in ElMassimo@8a581c1 with no explanation of why it was being removed.
Compatible with Vite 4. Previously, we were relying on Vite 5 using a unix hidden file path and `Dir.glob` ignoring those. This way actually ensures whatever the manifest path is, we protect it.
…-current-assets

20240704 do not clean compressed current assets
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants