Skip to content

fix(build): Ignore default-source.svg from image optimization - #40622

Merged
nickvergessen merged 1 commit into
masterfrom
bugfix/40410/ignore-default-source.svg-from-optimization
Sep 26, 2023
Merged

fix(build): Ignore default-source.svg from image optimization#40622
nickvergessen merged 1 commit into
masterfrom
bugfix/40410/ignore-default-source.svg-from-optimization

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Summary

image

Checklist

Signed-off-by: Joas Schilling <coding@schilljs.com>

@provokateurinprovokateurin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would prefer checking if the svg contains an <image/> tag to avoid this problem in other files as well. Right now this is the only file afaik, but maybe there will be more in the future

@nickvergessen
nickvergessen merged commit d5a8a97 into masterSep 26, 2023
@nickvergessen
nickvergessen deleted the bugfix/40410/ignore-default-source.svg-from-optimization branch September 26, 2023 07:14
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: themingtechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

build/image-optimization.sh needs adjusting

4 participants

@nickvergessen@miaulalala@bigcat88@provokateurin