8000 Release v2.17.0 by canaleal · Pull Request #2488 · devicons/devicon · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Release v2.17.0 #2488

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

Open
wants to merge 709 commits into
base: master
Choose a base branch
from
Open

Release v2.17.0 #2488

wants to merge 709 commits into from

Conversation

canaleal
Copy link
Member

A huge thanks to all our maintainers and contributors for making this release possible!

92 New/Updated Icons

14 New Features

gewaleelek and others added 30 commits January 17, 2025 14:37
new icon: clickhouse (original, plain)
new icon: cobol (original)
…ove-line.svg, removed padding on love-original
new icon: talos (original, plain)
new icon: gleam (original, plain)
…ecated-actions

ci(workflows): update deprecated actions
new icon: nats (original, plain)
@weh
Copy link
Contributor
weh commented Jul 2, 2025

angularjs

Looks like angular wasn't included in the build process. It was supposed to be in the last release:

* [Build preparation for release v2.16.0 #2105](https://github.com/devicons/devicon/pull/2105)

Maybe we can patch it in?

Yes WEH can 😁

Please see #2493 and test if the patch and local build worked.

What i did:

  • removed angular-plain and angular-plain-wordmark from icomoon.json to trigger the re-build
  • ran npm run build-icon on my local (linux) machine
  • checked the site with npm run dev and imported the icomoon.json in the icomoon app

canaleal and others added 2 commits July 6, 2025 17:49
@canaleal
Copy link
Member Author
canaleal commented Jul 6, 2025

Angular is fixed:
image

Copy link
Contributor
@weh weh left a comment

Choose a reason for hiding this comment

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

I verified the Angular fix and did some more random icon checks.
Guess we are ready 🚀

@weh weh requested a review from ReenigneArcher July 11, 2025 15:16
@ReenigneArcher
Copy link
Member

@weh as I'm new to this, is there a a dev site published somewhere where everything can be previewed or what is the process to review a release?

@Snailedlt
Copy link
Collaborator
Snailedlt commented Jul 11, 2025

@weh as I'm new to this, is there a a dev site published somewhere where everything can be previewed or what is the process to review a release?

@ReenigneArcher nope, but you can open it in vscode online. Just change the URL from .com to .dev like so: https://github.dev/devicons/devicon/pull/2488

From there just run npm install and npm run dev

@ReenigneArcher
Copy link
Member

These icons have off center SVGs. I checked every brand, but only the first icon for each.

  • antdesign
  • archlinux
  • contoa
  • emacs
  • fortran
  • ghost
  • gitpod
  • influxdb
  • jekyll
  • mobx
  • mongoose
  • nimble
  • norg
  • okta
  • portainer
  • purescript
  • reach
  • ros
  • sema
image

Maybe this is personal opinion, but should the go default be the "GO" icon instead of the mascot? For the font that's the case, but for the SVG it's reversed. In fact many icons do not have the same style as the default for the font and SVG... I'm not sure if that is intentional.
image

Should twitter be renamed to x, with an alias to twitter?

@canaleal
Copy link
Member Author

The SVGs are not properly center-aligned.
All the icons mentioned are from previous releases. Here's an example with Arch Linux:

image

@weh
Copy link
Contributor
weh commented Jul 11, 2025

Sorry for the spam, but i wanted to keep each icon separate. All of the listed icons had width and height attributes which needed to be removed. also many of them had multiple lines.

Now the next question is, if we can trigger a rebuild, or if i should try to "patch" the font files again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0