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

Asset Manager fixes #17531

Merged
merged 2 commits into from
Feb 25, 2025
Merged

Asset Manager fixes #17531

merged 2 commits into from
Feb 25, 2025

Conversation

Skrypt
Copy link
Contributor

@Skrypt Skrypt commented Feb 25, 2025

This fixes the issue where running yarn watch -n parcel-asset was not updating the asset in the destination folder. Also was failing silently doing so. Workaround was to delete manually these files in the destination folder.

  • Clean Parcel destination folder for each assets before running action. Parcel doesn't allow overwriting files so we need to keep them in separate folders to clean them before build or watch.

  • Fix min action issue that deletes the unminified file after minifying.

Clean Parcel destination folder for each assets before running action. Parcel doesn't allow overwriting files so we need to keep them in separate folders to clean them before build or watch.
Fix min action issue that deletes the unminified file after minifying.
@Skrypt Skrypt merged commit 1188808 into main Feb 25, 2025
8 checks passed
@Skrypt Skrypt deleted the skrypt/asset-manager-fixes branch February 25, 2025 21:32
@Skrypt
Copy link
Contributor Author

Skrypt commented Feb 25, 2025

Merging this as these are fixes for Parcel.

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.

1 participant