-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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 Axios icon #5991
Add Axios icon #5991
Conversation
Thanks for taking this one on, @SimonGolms 👍🏻 Would you mind having another look at it though, as I am seeing some differences throughout - such as this one in the top left - between the PR version (in red) and the source? |
@PeterShaggyNoble would be nice if this PR gets the 'hacktoberfest-accepted' label 😎 |
I am not seeing that difference with the newer version, so I will approve this but wait a few days before merging to see if @PeterShaggyNoble can review it. |
@sachinrajam, how should we proceed with this PR? |
We can merge it @dirien |
Hi @SimonGolms, we just need to get the Lint issue sorted! Then we are ready to merge! 😄 |
Thanks for pinging! The linter should be happy now 😊 |
Issue: #3821
Alexa rank: 315,678 https://www.alexa.com/siteinfo/axios-http.com
Github stars: 85.2k https://github.com/axios/axios
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
#5A29E4
directly from the mentioned file.Sorry for maybe taking over the issue ticket.... I am just checking the tech stack of some of my projects and adding missing logos to be able to use them later in the shields.io badges 🙃