Skip to content
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

Update GeeksforGeeks #5704

Merged
merged 2 commits into from May 18, 2021
Merged

Update GeeksforGeeks #5704

merged 2 commits into from May 18, 2021

Conversation

adamrusted
Copy link
Member

geeksforgeeks

Issue: #5653
Alexa rank: 283

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGO or SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

SVG pulled from homepage, hex pulled from SVG.

@github-actions github-actions bot added the update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both label May 15, 2021
@adamrusted adamrusted linked an issue May 17, 2021 that may be closed by this pull request
@PeterShaggyNoble
Copy link
Member

Looks good to me, @adamrusted. There's a duplicate point in each cut-out of the original, though and, while removing the one on the right leads to a slight distortion, it looks like the one on the left can be removed. If not, though, then this can be merged straight in.

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

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

Perfect, @adamrusted, nice work 👍🏻

@PeterShaggyNoble PeterShaggyNoble merged commit 279ccd6 into simple-icons:develop May 18, 2021
@adamrusted adamrusted deleted the Update/GeeksforGeeks branch May 18, 2021 14:39
ericcornelissen added a commit that referenced this pull request May 23, 2021
# New Icons

- Betfair (#5727)
- Chai (#5222)
- FMOD (#5720)
- ImageJ (#5494)
- Intigriti (#5733)
- MicroPython (#5681)
- Perforce (#5724)
- Spring Boot (#5687)
- tqdm (#5699)
- Weblate (#5688)
- Wwise (#5722)

# Updated Icons

- Azure Pipelines (#5248)
- Coinbase (#5712)
- FreeBSD (#5701)
- G2A (#5703)
- GeeksforGeeks (#5704)
- Geocaching (#5702)
- Ghost (#5739)
- Habr (#5713)
- Hackster (#5735)
- Helly Hansen (#5737)
- IBM (#5740)
- IMDb (#5700)
- Infosys (#5742)
- Nubank (#5734)
- PostgreSQL (#5715)
- Power BI (#5716)
- Viadeo (#5260)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update GeeksforGeeks icon
2 participants