Skip to content

Conversation

@Cherry
Copy link
Contributor

@Cherry Cherry commented May 2, 2025

Followup to #20446 and #17810 and #17350. Optimized more newly added SVGs.

Original size: ~59.64 MB, optimized size: ~51.2 MB. Saved ~8.44 MB

Copy link
Collaborator

@kodster28 kodster28 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Cherry-bot!
(we should probably have some built-in optimizations around this, tbh)

@kodster28 kodster28 merged commit 6bfb351 into cloudflare:production May 5, 2025
5 checks passed
@workers-devprod workers-devprod added the contribution [Holopin] Recognizes a docs contribution, big or small label May 5, 2025
@holopin-bot
Copy link

holopin-bot bot commented May 5, 2025

Congratulations @Cherry, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cmab4brob00270cjpc1ymug25

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

@Cherry
Copy link
Contributor Author

Cherry commented May 5, 2025

Thanks Cherry-bot! (we should probably have some built-in optimizations around this, tbh)

Agreed 😂

This script would be pretty simple to run and commit/PR once a month or whatever, so all it takes is a quick merge. Last I checked, Astro didn't do any optimisations on SVG files natively sadly.

daisyfaithauma pushed a commit that referenced this pull request May 7, 2025
daisyfaithauma pushed a commit that referenced this pull request May 8, 2025
Maddy-Cloudflare pushed a commit that referenced this pull request May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution [Holopin] Recognizes a docs contribution, big or small size/xl

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants