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.

Sign up for GitHub

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 Apache Druid #2874

Merged
merged 2 commits into from Apr 12, 2020
Merged

Add Apache Druid #2874

merged 2 commits into from Apr 12, 2020

Conversation

PeterShaggyNoble
Copy link
Member

Apache Druid

Issue: Closes #2857

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

Should we go with one of the colours from the website instead?

@PeterShaggyNoble PeterShaggyNoble added the new icon Issues or pull requests for adding a new icon label Mar 30, 2020
@PeterShaggyNoble PeterShaggyNoble mentioned this pull request Apr 1, 2020
Closed
17 tasks
ericcornelissen
ericcornelissen approved these changes Apr 12, 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.

SVG looks fine and the colour choice seems good to me 👍

Given that there has been no comment about the colour yet, I will be merging this as is. If the colour needs to be changed we can do that later.

@ericcornelissen ericcornelissen merged commit 08321ce into simple-icons:develop Apr 12, 2020
@ericcornelissen ericcornelissen mentioned this pull request Apr 12, 2020
Merged
ericcornelissen added a commit that referenced this pull request Apr 12, 2020
@ericcornelissen
Release 22 new icons and 5 updated icon (v2.9.0)
84a2e88 
# New icons

- CLiqz (#2858)
- DS Automobiles (#2840)
- Bloglovin (#2834)
- Atari (#2881)
- Qantas (#2839)
- CodersRank (#2876)
- Google Hangouts (#2843)
- PlayStation Vita (#2895)
- Hotels.com (#2894)
- Leaflet (#2868)
- Talend (#2867)
- Air France (#2891)
- Dior (#2903)
- Mozilla Thunderbird (#2906)
- Vultr (#2910)
- CocoaPods (#2905)
- Audacity (#2908)
- pixiv (#2829)
- Jameson (#2808)
- Audioboom (#2882)
- Amazon Fire TV (#2883)
- Apache Druid (#2874)

# Updated icon

- ResearchGate (#2888)
- Adobe Lightroom Classic (#2886)
- Adobe PhoneGap (#2886)
- Adobe Premiere (#2886)
- Adobe XD (#2886)
@PeterShaggyNoble PeterShaggyNoble deleted the apachedruid branch April 13, 2020 11:41
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.

Apache Druid
2 participants
@PeterShaggyNoble @ericcornelissen