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

new icon: flask (original, original-wordmark) #463

Merged
merged 3 commits into from
Jan 6, 2021
Merged

new icon: flask (original, original-wordmark) #463

merged 3 commits into from
Jan 6, 2021

Conversation

EnisMulic
Copy link
Contributor

Taken from #170

@Thomas-Boi Thomas-Boi added the bot:peek Use this label to trigger peek-bot. Remove and re-add the label to re-trigger label Jan 6, 2021
@amacado
Copy link
Member

amacado commented Jan 6, 2021

@EnisMulic maybe you can have a look at this #66? Maybe we can use them to enrich this pr?

@EnisMulic EnisMulic changed the title new icon: flask (plain) new icon: flask (original, original-wordmark) Jan 6, 2021
@amacado amacado added feature:icon Use this label for pull requests when a new icon is ready to be added to the collection 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 Jan 6, 2021
amacado
amacado requested changes Jan 6, 2021
View reviewed changes
Copy link
Member

@amacado amacado left a comment

Choose a reason for hiding this comment

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

Removing the id="Layer" and it's fine I think. Maybe @Thomas-Boi can have a look at the aliases?

new_icons
screenshot_0

screenshot_1

icons/flask/flask-original-wordmark.svg Outdated Show resolved Hide resolved
@amacado amacado mentioned this pull request Jan 6, 2021
Closed
@EnisMulic EnisMulic requested a review from amacado January 6, 2021 16:53
@Thomas-Boi
Copy link
Member

@amacado I think @EnisMulic is waiting for us to decide whether to keep using the aliases attribute in our repo. I've opened a discussion on this topic, which you can see here

That being said, @EnisMulic got the aliases right. Everything looks good and we can just merge this PR. If we need to fix the aliases, we'll have to do it for the whole repo later anyway.

@amacado amacado merged commit 5c0f823 into devicons:develop Jan 6, 2021
@EnisMulic EnisMulic deleted the feature/icon/flask branch January 6, 2021 22:42
@amacado amacado mentioned this pull request Jan 19, 2021
Merged
amacado added a commit that referenced this pull request Jan 20, 2021
@amacado @tylensthilaire
@EnisMulic @Thomas-Boi @Th1nkK1D @moghya @eldadfux @github-actions
Release 2.9.0 (#485)
9c6bfdb 
* Create eleventy-original.svg

* Create eleventy-plain.svg

* Update devicon.json

* Update eleventy-plain.svg

* Update eleventy-original.svg

* new icon: sqlalchemy (plain, original, original-workmark)

* Add font aliase for sqlalchemy

* new icon: microsoftsqlserver (plain, plain-wordmark) (#427)

* new icon sqlserver (plain, plain-woodmark)

* Update icons/sqlserver/sqlserver-plain-wordmark.svg

Co-authored-by: Clemens Bastian <[email protected]>

* Update icons/sqlserver/sqlserver-plain.svg

Co-authored-by: Clemens Bastian <[email protected]>

* Update devicon.json

Co-authored-by: Clemens Bastian <[email protected]>

* Update devicon.json

Co-authored-by: Clemens Bastian <[email protected]>

* Rename sqlserver icons to microsoftsqlserver

Co-authored-by: Clemens Bastian <[email protected]>

* new icon: kubernetes (plain, plain-wordmark) (#424)

* new icon: kubernetes (plain, plain-wordmark)

* Update icons/kubernetes/kubernetes-plain.svg

Co-authored-by: Clemens Bastian <[email protected]>

* Update icons/kubernetes/kubernetes-plain-wordmark.svg

Co-authored-by: Clemens Bastian <[email protected]>

Co-authored-by: Clemens Bastian <[email protected]>

* new icon: rocksdb (plain) (#423)

* new icon: rocksdb (plain)

* Update icons/rocksdb/rocksdb-plain.svg

Co-authored-by: Clemens Bastian <[email protected]>

Co-authored-by: Clemens Bastian <[email protected]>

* new icon: googlecloud (original, original-wordmark, plain, plain-wordmark) (#428)

* new icon: googlecloud (plain, plain-wordmark, original, original-wordmark)

* Update icons/googlecloud/googlecloud-plain.svg

Co-authored-by: Clemens Bastian <[email protected]>

* Update icons/googlecloud/googlecloud-original-wordmark.svg

Co-authored-by: Clemens Bastian <[email protected]>

* Update icons/googlecloud/googlecloud-original.svg

Co-authored-by: Clemens Bastian <[email protected]>

* Fix google cloud icons

* Fix googlecloud original

* Remove fill redefinition from google cloud original

* Fix googlecloud alignment (Test)

* Fix googlecloud alignment (test)

* Fix googlecloud alignment (test)

Co-authored-by: Clemens Bastian <[email protected]>

* new icon: objectivec (plain) (#425)

* new icon: objectivec (plain)

* Update icons/objectivec/objectivec-plain.svg

Co-authored-by: Clemens Bastian <[email protected]>

Co-authored-by: Clemens Bastian <[email protected]>

* new icon uwsgi (original, plain)

* Cleanup uwsgi icons

* Update devicon.json

Co-authored-by: Thomas Bui <[email protected]>

* Add adobe XD line icon

* Add xd-plain and eps file

* Clean up xd svg file

* Update devicon.json

* new icon: firebase (plain, plain-wordmark) (#461)

* add icon Firebase (plain, plain-wordmark), Closes #204

* new icon: firebase (plain, plain-wordmark)

* update icons

* Add plain-wordmark to firebase fonts

* new icon: flask (original, original-wordmark) (#463)

* Cleanup flask icon

* Improve flask icons

* Update icons/flask/flask-original-wordmark.svg

Co-authored-by: Clemens Bastian <[email protected]>

Co-authored-by: moghya <[email protected]>
Co-authored-by: Clemens Bastian <[email protected]>

* Update devicon.json

* Fixed an error with devicon.json

* Added check script for the icons and fix icons with fill or viewBox issues (#460)

* Added script to check svg fill and viewBox

* Fix cucumber-plain-wordmark

* Fix various fill and viewBox issues in svgs

* Added check for height and width attr

* Added check_svgs workflow

* Fix an issue where the error is not log properly

* Added on push for testing

* Updated trigger so it now runs whenever PR is update

* Added sleep to script to make logs nicer

* Added script that create env var

* Updated the github_env to accomodate ubuntu

* Change format of log and allow filehandler to return Path

* Updated logging messages

* Updated refs for the checkout action

* Make logging nicer

* Updated fix messaging so it's more clear

* fix icons: icons/cucumber/cucumber-plain-wordmark.svg, icons/intellij/intellij-plain-wordmark.svg, icons/jenkins/jenkins-plain.svg, icons/twitter/twitter-original.svg, icons/yunohost/yunohost-plain.svg

* Fix the ref issue of the checkout action and sqlalchemy  (#472)

* Remove head_ref from checkout action

* Fixed the svg errs in sqlalchemy

* Create a monthly script that checks all svgs

* Python now print traceback

* Updated file names in check_svgs_on_pr

* Remove deprecated and unused svg syntax

Co-authored-by: Clemens Bastian <[email protected]>

* Remove deprecated and unused svg syntax

Co-authored-by: Clemens Bastian <[email protected]>

* Update svg fill, remove unused class

* Change comment action to a new action

* Add guideline about squash merging

As the result of discussion #470 update guidelines with squash merging

* Redo the workflow archs so we can comment on pr

* Moved check_svgs_monthly to draft

* Added working workflow file

* Changed to file read action

* new icon: appwrite (plain, original, wordmark) (#371)

* Added Appwrite icon

* Fixed wrong JSON input

* Fixed JSON style

* Fixed indent

* Update devicon.json

Co-authored-by: Clemens Bastian <[email protected]>

* Update devicon.json

Co-authored-by: Clemens Bastian <[email protected]>

* Update devicon.json

Co-authored-by: Clemens Bastian <[email protected]>

Co-authored-by: Clemens Bastian <[email protected]>

* bot:build new icons, icomoon.json and devicon.css (#486)

* fix bash-plain (#451) (#453)

Co-authored-by: Enis Mulić <[email protected]>

* Built new icons, icomoon.json and devicon.css

Co-authored-by: Clemens Bastian <[email protected]>
Co-authored-by: Enis Mulić <[email protected]>
Co-authored-by: amacado <[email protected]>

* Removed 'default fall back icon' from build_icons.yml

Co-authored-by: Tylen St Hilaire <[email protected]>
Co-authored-by: Enis Mulić <[email protected]>
Co-authored-by: Thomas Bui <[email protected]>
Co-authored-by: Withee Poositasai <[email protected]>
Co-authored-by: moghya <[email protected]>
Co-authored-by: Thomas Bui <[email protected]>
Co-authored-by: Eldad A. Fux <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: amacado <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Thomas-Boi Thomas-Boi Awaiting requested review from Thomas-Boi

@amacado amacado Awaiting requested review from amacado

Assignees
No one assigned
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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@EnisMulic @amacado @Thomas-Boi @moghya