Repository navigation
chore: use simple-icons for the tangled icon - #3321
Conversation
Simple Icons now includes Tangled, so use that instead of the custom copy of the logo. Also adds the icon to the providers table in the docs.
Thanks for opening this pull request! 🎉We really appreciate you taking the time to contribute, @Tobbe. A maintainer will take a look as soon as they can. In the meantime, please make sure that:
If anything needs adjusting we'll leave comments here. Thanks again! |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The Tangled icon update appears ready to merge after normal checks. Pre-merge checks |
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
trueberryless
left a comment
There was a problem hiding this comment.
Amazing!
Thanks for the PR 🚀
Simple Icons now includes Tangled, so use that instead of the custom copy of the logo. Also adds the icon to the providers table in the docs.
🔗 Linked issue
N/A
🧭 Context
Using the official simple-icons version of the Tangled logo
📚 Description
Instead of using a custom/vendored version of the Tangled logo, this PR switches over to the Tangled icon that's included in the simple-icons icon set. Most other forges already use an icon from the existing set, so this brings Tangled in line with those.
Here's the icon on the simpleicons web page https://simpleicons.org/?q=tangled