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

Add support for --skip-if-larger option of pngquant #140

Merged
merged 7 commits into from
Apr 22, 2021

Conversation

MrPropre
Copy link
Contributor

Check #139 for more details

@MrPropre
Copy link
Contributor Author

The CI build failed with error: Process errored with error: unknown option '--disable={cleanupIDs,removeViewBox}'
It seems that in SVGO 2.0, --enable and --disable flags are removed.
svg/svgo#1356 (comment)

@MrPropre
Copy link
Contributor Author

For now, I have fixed the SVGO version to 1.3.2 because 2.x is not backward compatible.

@freekmurze
Copy link
Member

I'm thinking that this will break it for people that are on v2 of pngquant

@MrPropre
Copy link
Contributor Author

MrPropre commented Apr 20, 2021

@freekmurze I think you are talking about SVGO. Feel free to propose another solution for SVGO 2 users. This PR is initially for pngquant but tests did not pass because of SVGO 2. I added a note about SVGO 2 in the readme file.

@freekmurze freekmurze merged commit db402f4 into spatie:master Apr 22, 2021
@freekmurze
Copy link
Member

Thank you!

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.

None yet

2 participants