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

new icon: hadoop (original, original-wordmark, plain, plain-wordmark) #1488

Merged
merged 16 commits into from
Dec 10, 2022

Conversation

MAGistR-bit
Copy link
Contributor

Apache Hadoop

Opened the pull request (again)

amacado and others added 14 commits March 22, 2022 22:31
Fix issue with npm publish script
Build preparation for release v2.15.1
The following actions were performed:

* Updated devicon.json

* Various versions of the icon have been added
The following actions were performed:

* Updated devicon.json

* Various versions of the icon have been added
The following changes have been made:

* Updated the json file

* Renamed the file

* Deleted the file (hadoop-original-wordmark.svg)
The following changes have been made:

* Optimized the images (SVG)

* Merged the paths (for plain and line icons) into one
The following changes have been made:

* Deleted extra files

* Updated the file
Co-authored-by: Jørgen Kalsnes Hagen <[email protected]>
Edited the color of the image
@MAGistR-bit
Copy link
Contributor Author

@Snailedlt I have opened a new pull request (PR).

@Snailedlt Snailedlt added the bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger label Oct 30, 2022
@github-actions
Copy link
Contributor

Hi!

I'm the check-bot and we have some issues with your PR:

SVG Error in 'hadoop-original.svg':
- SVG contains `stroke` property. This will get ignored by Icomoon. Please convert them to fills.

Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@github-actions
Copy link
Contributor

Hi there,

I'm Devicons' Peek Bot and I just peeked at the icons that you wanted to add using icomoon.io.

Here are the SVGs as intepreted by Icomoon when we upload the files:
Imgur Images

Here are the zoomed-in screenshots of the added icons as SVGs:
Imgur Images

Here are the icons that will be generated by Icomoon:
Imgur Images

Here are the zoomed-in screenshots of the added icons as icons:
Imgur Images

Here are the colored versions:
Imgur Images

The maintainers will now check for:

  1. The number of Glyphs matches the number of SVGs that were selected.
  2. The icons (second group of pictures) look the same as the SVGs (first group of pictures).
  3. The icons are of high quality (legible, matches the official logo, etc.)

In case of font issues, it might be caused by Icomoon not accepting strokes in the SVGs. Check this doc for more details and fix the issues as instructed by Icomoon and update this PR once you are done.

Thank you for contributing to Devicon! I hope that your icons are accepted into the repository.

Note: If the images don't show up, it has been autodeleted by Imgur after 6 months due to our API choice.

Cheers,
Peek Bot 😊

Snailedlt
Snailedlt previously approved these changes Oct 30, 2022
Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

Looks good! ✔️
Thank you for your contribution, and your patience! 💯

PS: The bot message saying that 'stroke' should be replaced by 'fill' can be ignored, since the hadoop-original icon isn't used on icomoon as explained in this issue: #1325

@Snailedlt Snailedlt added request:icon When requesting a new icon to be added to the collection or a refactor use this label in your issue feature:icon Use this label for pull requests when a new icon is ready to be added to the collection and removed request:icon When requesting a new icon to be added to the collection or a refactor use this label in your issue labels Oct 30, 2022
@Snailedlt Snailedlt added the hacktoberfest-accepted Accepted to be counted towards Hacktoberfest label Oct 31, 2022
devicon.json Outdated Show resolved Hide resolved
@lunatic-fox
Copy link
Contributor

Remastered icons

original old

original new


plain old

plain new


New icons

original-wordmark

plain-wordmark

Source: https://svn.apache.org/repos/asf/comdev/project-logos/originals/hadoop.svg

@Snailedlt Snailedlt added bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger and removed bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger labels Dec 7, 2022
@Snailedlt
Copy link
Collaborator

Seems like this PR has become stale.

@lunatic-fox Feel free to commit the icons you suggested here, and take over the PR :)

@devicons devicons deleted a comment from github-actions bot Dec 7, 2022
@lunatic-fox lunatic-fox changed the title new icon: hadoop (original, plain) new icon: hadoop (original, original-wordmark, plain, plain-wordmark) Dec 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2022

Hi!

I'm the check-bot and we have some issues with your PR:

SVG Error in 'hadoop-original-wordmark.svg':
- SVG contains `stroke` property. This will get ignored by Icomoon. Please convert them to fills.

Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@Snailedlt
Copy link
Collaborator

@lunatic-fox Ignore this message
ref: #1488 (review)

Snailedlt
Snailedlt previously approved these changes Dec 7, 2022
Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

LGTM! 💯

@lunatic-fox
Copy link
Contributor

@lunatic-fox Ignore this message ref: #1488 (review)

Better remove it anyway, don't you think?

@Snailedlt
Copy link
Collaborator

@lunatic-fox Ignore this message ref: #1488 (review)

Better remove it anyway, don't you think?

No need imo. Well if it gets the file size down without changing the icon, then sure, otherwise we can just keep it :)

@lunatic-fox
Copy link
Contributor

lunatic-fox commented Dec 7, 2022

@lunatic-fox Ignore this message ref: #1488 (review)

Better remove it anyway, don't you think?

No need imo. Well if it gets the file size down without changing the icon, then sure, otherwise we can just keep it :)

It's a little stroke from the original icon. Don't know why is there, because it doesn't appear, but in order to prevent any resize bug (like it for some reason don't resize with the icon) I'll remove it really fast. 🙂

@lunatic-fox lunatic-fox added bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger and removed bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger labels Dec 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2022

Hi there,

I'm Devicons' Peek Bot and I just peeked at the icons that you wanted to add using icomoon.io.

Here are the SVGs as intepreted by Icomoon when we upload the files:
Imgur Images

Here are the zoomed-in screenshots of the added icons as SVGs:
Imgur Images Imgur Images

Here are the icons that will be generated by Icomoon:
Imgur Images

Here are the zoomed-in screenshots of the added icons as icons:
Imgur Images Imgur Images

Here are the colored versions:
Imgur Images Imgur Images

The maintainers will now check for:

  1. The number of Glyphs matches the number of SVGs that were selected.
  2. The icons (second group of pictures) look the same as the SVGs (first group of pictures).
  3. The icons are of high quality (legible, matches the official logo, etc.)

In case of font issues, it might be caused by Icomoon not accepting strokes in the SVGs. Check this doc for more details and fix the issues as instructed by Icomoon and update this PR once you are done.

Thank you for contributing to Devicon! I hope that your icons are accepted into the repository.

Note: If the images don't show up, it has been autodeleted by Imgur after 6 months due to our API choice.

Cheers,
Peek Bot 😊

@Snailedlt Snailedlt merged commit fe85091 into devicons:develop Dec 10, 2022
@Snailedlt Snailedlt mentioned this pull request Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger feature:icon Use this label for pull requests when a new icon is ready to be added to the collection hacktoberfest-accepted Accepted to be counted towards Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants