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

feat: add svgo to minify sprite, re-add hashing #21

Merged
merged 7 commits into from
Apr 4, 2024

Conversation

mcansh
Copy link
Owner

@mcansh mcansh commented Apr 4, 2024

available for testing via experimental npm tag. let me know if theres any issues outside of the added logging :)

mcansh added 2 commits April 4, 2024 12:38
Signed-off-by: Logan McAnsh <logan@mcan.sh>
x
Signed-off-by: Logan McAnsh <logan@mcan.sh>
Copy link

changeset-bot bot commented Apr 4, 2024

🦋 Changeset detected

Latest commit: 6ce35b3

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

This PR includes changesets to release 2 packages
Name Type
@mcansh/vite-svg-sprite-plugin Patch
vite-no-remix 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

mcansh added 4 commits April 4, 2024 13:20
Signed-off-by: Logan McAnsh <logan@mcan.sh>
Signed-off-by: Logan McAnsh <logan@mcan.sh>
Signed-off-by: Logan McAnsh <logan@mcan.sh>
Signed-off-by: Logan McAnsh <logan@mcan.sh>
@sergiodxa
Copy link

Can confirm this fixes #20

@mcansh
Copy link
Owner Author

mcansh commented Apr 4, 2024

Can confirm this fixes #20

sweeeeeet

Signed-off-by: Logan McAnsh <logan@mcan.sh>
@mcansh mcansh force-pushed the logan/sprite-hash-again branch from c15d70d to 6ce35b3 Compare April 4, 2024 17:41
@mcansh mcansh marked this pull request as ready for review April 4, 2024 17:47
@mcansh mcansh merged commit b4955f5 into main Apr 4, 2024
1 check passed
@mcansh mcansh deleted the logan/sprite-hash-again branch April 4, 2024 19:59
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.

2 participants