Skip to content

Conversation

@nyqykk
Copy link
Member

@nyqykk nyqykk commented Oct 25, 2024

Summary

MF assets are loaded over the network, which means they will not be compressed by the project. Therefore, minifying them is necessary.

@nyqykk nyqykk changed the title fix: mf assets should be compressed fix: mf assets should be minified Oct 25, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 25, 2024

CodSpeed Performance Report

Merging #328 will not alter performance

Comparing fix/compress-mf-assets (7f7a4cc) with main (a59551a)

Summary

✅ 5 untouched benchmarks

@netlify
Copy link

netlify bot commented Oct 25, 2024

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit a161b55
🔍 Latest deploy log https://app.netlify.com/sites/rslib/deploys/671b5374d2702600082ee4e1
😎 Deploy Preview https://deploy-preview-328--rslib.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@fi3ework fi3ework changed the title fix: mf assets should be minified fix: "mf" format output should be minified by default Oct 25, 2024
@fi3ework fi3ework enabled auto-merge (squash) October 25, 2024 09:55
@fi3ework fi3ework merged commit 3a68db0 into main Oct 25, 2024
18 checks passed
@fi3ework fi3ework deleted the fix/compress-mf-assets branch October 25, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants