Skip to content

Conversation

Thomas-Boi
Copy link
Member

Removed the strokes in the threejs icons as seen in #788. You can see the new bot-peek's scan for this icon here

@Thomas-Boi Thomas-Boi added bot:peek Trigger peek-bot. Remove and re-add the label to re-trigger feature:icon PR when a new icon is ready to be added to the collection labels Aug 10, 2021
@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 is the result below (top left):

Imgur Images

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

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

The maintainers will now take a look at it and decide whether to merge your PR.

Thank you for contributing to Devicon! I hope everything works out and your icons are accepted into the repo.

Cheers,
Peek Bot 😊

Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Thank you, @Thomas-Boi, for all your help in fixing all these icons that had strokes. 😄👍🎉

@Thomas-Boi
Copy link
Member Author

Thank you, @Thomas-Boi, for all your help in fixing all these icons that had strokes. 😄👍🎉

No problem! Watch out though, there might be more. I'm running a check for all of them one more time

@Thomas-Boi Thomas-Boi merged commit f90aae3 into develop Aug 12, 2021
@Thomas-Boi Thomas-Boi deleted the thomas/icons/threejs branch August 12, 2021 22:32
@amacado amacado mentioned this pull request Aug 14, 2021
GCHQDeveloper926 pushed a commit to GCHQDeveloper926/devicon that referenced this pull request Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:peek Trigger peek-bot. Remove and re-add the label to re-trigger feature:icon PR when a new icon is ready to be added to the collection
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants