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

Upgrade SVGO to v3 #76

Closed
wants to merge 2 commits into from
Closed

Conversation

TheOtterlord
Copy link

I've gone through the changelog for SVGO v3, and tested my changes with a copy of the demo & an up-to-date Astro v1.8.0 project that used astro-compress

@changeset-bot
Copy link

changeset-bot bot commented Dec 31, 2022

🦋 Changeset detected

Latest commit: 547d523

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
astro-icon Minor
www Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Dec 31, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-astroicon ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 25, 2023 2:48pm
astroicon ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 25, 2023 2:48pm

Copy link

@rdwz rdwz left a comment

Choose a reason for hiding this comment

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

Looks good to me

@natemoo-re pls have a look at pr

@dkotik
Copy link

dkotik commented Mar 23, 2023

Please merge :-)

@stramel
Copy link
Collaborator

stramel commented May 25, 2023

I think this would be okay as a minor bump for the existing package code. The v1 branch already has the SVGO v3 integrated.

Are there any side-effects from updating to v3?

@stramel
Copy link
Collaborator

stramel commented Jun 29, 2023

Thanks for this PR! At this time we're going to forego this and focus on the v1 since this has a workaround and we have published a next version of astro-icon that includes using svgo@v3

@stramel stramel closed this Jun 29, 2023
a-game added a commit to a-game/a-game.github.io that referenced this pull request Oct 7, 2023
# Note
svgo 2.8.0 is not actually used by this repo, however it is used by
astro-icon and the build fails unless it has that exact version
[according to this](natemoo-re/astro-icon#76)

Remove it once astro-icon is updated.
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.

Error: Unknown builtin plugin "cleanupIDs" specified. Updating to svgo@3 causes issues
6 participants