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

feat(reporter): add wasm to the compressible assets regex #19085

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yurivish
Copy link

Wasm files are often highly compressible. It would be nice to show their compressed size in Vite builds. This PR is a one-line change that should show their compressed size after the build is complete.

I found an earlier PR (#12485) that made this change for some other asset types, and it had a comment that sounded open to the addition of wasm to the list:

Feel free to add wasm to COMPRESSIBLE_ASSETS_RE here.

Description

Allows .wasm files to show their compressed size in the Vite build results

Wasm files are often highly compressible and it would be nice to show their compressed size in Vite builds. 

I found an earlier PR (vitejs#12485) that made this change for some other asset types, and a [comment](vitejs#12485 (comment)) that sounded open to the addition of wasm to the list, which I've added.
@yurivish yurivish changed the title Add wasm to the compressible assets regex feat: Add wasm to the compressible assets regex Dec 27, 2024
@sapphi-red sapphi-red changed the title feat: Add wasm to the compressible assets regex feat: add wasm to the compressible assets regex Dec 28, 2024
@sapphi-red sapphi-red changed the title feat: add wasm to the compressible assets regex feat(reporter): add wasm to the compressible assets regex Dec 28, 2024
@sapphi-red sapphi-red added the p2-nice-to-have Not breaking anything but nice to have (priority) label Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2-nice-to-have Not breaking anything but nice to have (priority)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants