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

update icon: tensorflow (original-wordmark, line-wordmark) #1168

Merged
merged 2 commits into from
Apr 30, 2022
Merged

update icon: tensorflow (original-wordmark, line-wordmark) #1168

merged 2 commits into from
Apr 30, 2022

Conversation

maltejur
Copy link
Contributor

@maltejur maltejur commented Apr 30, 2022 •
edited
Loading

These icons for some reasons have a inline font included which isn't even used, so this PR removes it.

Edit: Since the check-bot complained, I completely cleaned up the icons.

@github-actions
Copy link
Contributor

Hi!

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

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

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

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

SVG Error in 'tensorflow-line-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 😄

@amacado amacado merged commit 3cc4764 into devicons:develop Apr 30, 2022
Snailedlt pushed a commit to Snailedlt/devicon that referenced this pull request Oct 15, 2022
@maltejur @Snailedlt
update icon: tensorflow (original-wordmark, line-wordmark) (devicons#…
832bd8b 
…1168)

* remove inline font from tensorflow wordmark icons

* clean up tensorflow icons even more
Snailedlt pushed a commit to Snailedlt/devicon that referenced this pull request Oct 15, 2022
@maltejur @Snailedlt
update icon: tensorflow (original-wordmark, line-wordmark) (devicons#…
9929b95 
…1168)

* remove inline font from tensorflow wordmark icons

* clean up tensorflow icons even more
Snailedlt pushed a commit to Snailedlt/devicon that referenced this pull request Oct 15, 2022
@maltejur @Snailedlt
update icon: tensorflow (original-wordmark, line-wordmark) (devicons#…
e94d0b2 
…1168)

* remove inline font from tensorflow wordmark icons

* clean up tensorflow icons even more
@Snailedlt Snailedlt added the feature:icon Use this label for pull requests when a new icon is ready to be added to the collection label Nov 5, 2023
@Snailedlt Snailedlt mentioned this pull request Feb 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amacado amacado amacado approved these changes

Assignees
No one assigned
Labels
feature:icon Use this label for pull requests when a new icon is ready to be added to the collection
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@maltejur @amacado @Snailedlt