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 Wasmer icon #7646
Add Wasmer icon #7646
Conversation
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.
Thanks for your contribution, @syrusakbary! 🙏
The icon is perfectly identical with the source SVG image (almost identical data path even). 👍
I have only a few minor requests.
_data/simple-icons.json
Outdated
"title": "wasmer", | ||
"hex": "4946DD", | ||
"source": "https://github.com/wasmerio/wasmer.io/blob/0d425f5b4ace56496e75278e304f54492c46adde/public/images/icon.svg", | ||
"guidelines": "https://wasmer.io/" |
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 can't find the logo guidelines on this website. Unless I've overlooked it somewhere, please delete this field.
Co-authored-by: Sigurd Spieckermann <2206639+sisp@users.noreply.github.com>
I just fixed the suggestions @sisp . Thanks for reviewing and looking forward to having the PR merged! |
@syrusakbary The tests are failing beause of the trailing comma after the source URL. When you remove it, they should pass. |
/ping @syrusakbary 😉 |
@Sisip Thanks for following up :) |
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 now! 🎉
Thanks again for your contribution, @syrusakbary! 🙏
Issue: closes #7643
Similarweb rank: 291,242 (https://www.similarweb.com/website/wasmer.io)
Github Stars: 12,700 (https://github.com/wasmerio/wasmer/)
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description