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

Jump to bottom

Add Ansys #3211

Merged
merged 5 commits into from Jun 16, 2020
Merged

Add Ansys #3211

merged 5 commits into from Jun 16, 2020

Conversation

mondeja
Copy link
Member

@mondeja mondeja commented Jun 12, 2020 •
edited

ansys

Issue: #1984

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

  • File from website header.
  • Color #FFB71B from logo and website stylesheet.
  • Alexa Rank: ~31k

@ericcornelissen ericcornelissen added the new icon Issues or pull requests for adding a new icon label Jun 14, 2020
@runxel runxel mentioned this pull request Jun 16, 2020
Open
14 tasks
ericcornelissen
ericcornelissen suggested changes Jun 16, 2020
View reviewed changes
Copy link
Contributor

@ericcornelissen ericcornelissen left a comment •
edited

Choose a reason for hiding this comment

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

All looks good to me, but given that Ansys does have brand guidelines, let's update the source to that link 😃

Also, no need to keep updating your PRs with the develop branch manually, once we reviewed it and all is well we will do that for you

_data/simple-icons.json Outdated Show resolved Hide resolved
ericcornelissen
ericcornelissen approved these changes Jun 16, 2020
View reviewed changes
Copy link
Contributor

@ericcornelissen ericcornelissen left a comment

Choose a reason for hiding this comment

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

Thanks for the update @mondeja, I'll be merging this once the build succeeds.

@ericcornelissen ericcornelissen merged commit 463450c into simple-icons:develop Jun 16, 2020
@github-actions github-actions bot mentioned this pull request Jun 21, 2020
Merged
github-actions bot added a commit that referenced this pull request Jun 21, 2020
@github-actions
Release 5 new icons (v2.18.0)
7b0e962 
# New Icons

- Ansys (#3211)
- Google News (#2790)
- Mini (#3210)
- Shikimori (#3179)
- Trove (#3069)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericcornelissen ericcornelissen ericcornelissen approved these changes

Assignees
No one assigned
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mondeja @ericcornelissen