new icon: excel (plain)

[Thomas-Boi]

Taken from #116. Work was done by @Pmz64.

[github-actions]

Hi there,

I'm Devicons' Peek Bot and it seems we've ran into a problem (sorry!).

Please double check and fix the possible issues below:

• Your svgs are named and added correctly to the /icons folder as seen here.
• Your icon information has been added to the devicon.json as seen here
• Your PR title follows the format seen here

I will retry once everything is fixed. If I still fail (sorry!) or there are other erros, the maintainers will investigate.

Best of luck,
Peek Bot ☺️

[github-actions]

Hi there,

I'm Devicons' Peek Bot and it seems we've ran into a problem (sorry!).

Please double check and fix the possible issues below:

• Your svgs are named and added correctly to the /icons folder as seen here.
• Your icon information has been added to the devicon.json as seen here
• Your PR title follows the format seen here

I will retry once everything is fixed. If I still fail (sorry!) or there are other erros, the maintainers will investigate.

Best of luck,
Peek Bot ☺️

[github-actions]

Hi there,

I'm Devicons' Peek Bot and it seems we've ran into a problem (sorry!).

Please double check and fix the possible issues below:

• Your svgs are named and added correctly to the /icons folder as seen here.
• Your icon information has been added to the devicon.json as seen here
• Your PR title follows the format seen here

I will retry once everything is fixed. If I still fail (sorry!) or there are other erros, the maintainers will investigate.

Best of luck,
Peek Bot ☺️

[github-actions]

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 right):

Here are the zoomed-in screenshots of the added icons:

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 😊

[amacado]

I'm not sure if Microsoft Excel should be part of this iconset. I unterstand that you can script within excel (macros/vb) but I'm not sure if it fit's within our guidelines. I would like to open a discussion about it.

[Thomas-Boi]

I was a bit iffy on this too but apparently, many businesses like small accounting or insurance firms still use Excel as a database of sort. Furthermore, I've heard learning VBA is a huge advantage => VBA runs in Excel so it makes Excel like an "editor" for VBA.

[Panquesito7]

Well, many developers use Excel, but its purpose is not mainly for developers. I'm not really sure what to do, but I guess we can remove it. There's no harm if we keep it or remove it. 🙂

[Panquesito7]

The icon itself looks good, though. 👍

[Thomas-Boi]

Let's open this up in Discussion and see what people think. I'm still not sure how to deal with this specific icon.

[cseelus]

Well, Excel is the reason many mere mortals have done reactive programming, probably without even realizing it: https://branch-blog.qlik.com/what-is-reactive-programming-a1e82cf28575

[Thomas-Boi]

Since there's not a lot of strong support for Excel, I think we can close this PR for now. If we ever need to get the icon (see someone make a good case for it), we can reopen this.

[Thomas-Boi]

I'll keep a local copy of the branch