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.

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

Removal of Alitalia #8310

Merged
merged 2 commits into from May 25, 2023
Merged

Removal of Alitalia #8310

merged 2 commits into from May 25, 2023

Conversation

ripzerotwo
Copy link
Contributor

Issue: closes #8309

Description

Removal of Alitalia icon as they are now defunct.

@service-paradis service-paradis added breaking change Issues or pull requests that include a breaking change and scheduled for the next major release removal request labels Feb 7, 2023
@service-paradis service-paradis added this to the v9.0.0 milestone Feb 7, 2023
@dirien
Copy link
Member

dirien commented Feb 19, 2023

@service-paradis do we need to remove the icons of defunct brands? I would not do it, as they are not really bothering much and maybe folks use it. So why an breaking change? I understand for any legal reasons though.

@service-paradis
Copy link
Member

service-paradis commented Feb 20, 2023

@dirien We always remove icons for defunct brands at major release as mentioned in https://github.com/simple-icons/simple-icons/blob/develop/VERSIONING.md. I labeled this as breaking change to be sure we do not currently merge this before the next major release. You can take a look at #3912 that discussed this subject a little bit.

@dirien
Copy link
Member

dirien commented Feb 20, 2023

@service-paradis ahhh okay! Did not know this! 😅 makes senes!

@service-paradis
Copy link
Member

haha, no problem :)

@mondeja mondeja merged commit 60c2fa6 into simple-icons:develop May 25, 2023
3 checks passed
mondeja added a commit that referenced this pull request May 28, 2023
# New Icons

- Ameba (#8655)
- Compiler Explorer (#8646)
- Frontify (#8634)
- Lightning (#8415)
- Misskey (#8412)
- MonoGame (#8614)
- MUBI (#8657)
- Payload CMS (#8633)
- Trailforks (#8635)

# Updated Icons

- Roblox (#8676)

# Removed Icons

- Alitalia (#8310)
- Linode (#8549)
- MonoGames (#8614)
- PyTorch Lightning (#8415)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Issues or pull requests that include a breaking change and scheduled for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Alitalia
4 participants