-
Notifications
You must be signed in to change notification settings - Fork 8k
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
Remove unused assets #14664
Remove unused assets #14664
Conversation
Hi @queengooborg, should we always check assets under the folders that contains changed documents in PR test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that the firefox_logo.png
under this folder is also not used anymore. Did filecheck tool misses this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@queengooborg seems good for RU, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for ko
. Thank you have a nice day :)
for yari-content-ko team
translated-content repo에서 모든 이미지 파일들은 이제 추가하지 않아도 됩니다. 참고 부탁드립니다. ( 참고 ) (cc. @mdn/yari-content-ko)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've removed one more unused image in zh-cn folder. Thank you @queengooborg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for pt-BR
This PR removes unused assets as found by
yarn filecheck
.