Skip to content

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

Merged
patak-cat merged 1 commit intovitejs:mainfrom
yurivish:patch-1
Jan 23, 2025
Merged

feat(reporter): add wasm to the compressible assets regex#19085
patak-cat merged 1 commit intovitejs:mainfrom
yurivish:patch-1

Conversation

@yurivish
Copy link
Copy Markdown
Contributor

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
@sapphi-red sapphi-red added this to the 6.1 milestone Jan 7, 2025
@patak-cat patak-cat merged commit ce84142 into vitejs:main Jan 23, 2025
moonlitusun pushed a commit to moonlitusun/vite that referenced this pull request May 25, 2025
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.

4 participants