-
Notifications
You must be signed in to change notification settings - Fork 522
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
removes nazi jacket sprite from suits.dmi #6389
Conversation
Maybe you'd, you know, remove many other unused sprites in there as well? Which is like half of them, that file probably never seen cleaning |
yea maybe, i just saw this one as glaringly worth removal and pressing "delete" when i know it's in there from prior looking is really easy since i just got home from work and don't want to validate and check every other file in there |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IDB is either dead or explicitly filters out that name refusing to run it. But checked out the branch regardless and I only see 1 sprite removed.
IconDB is down for the moment it appears. But I see no other alterations either. |
# About the pull request removes nazi jacket sprite from suits.dmi # Explain why it's good for the game 1. most modern SS13 codebases have specific stipulations in contributing not to include nazi iconography or themes; need i point out we probably best not keep it in the repo either 2. unused anywhere # Testing Photographs and Procedure ![image](https://github.com/cmss13-devs/cmss13/assets/29965103/829d03b7-e655-4976-90d2-1da66942905e) # Changelog :cl: del: removes the nazi jacket sprite from suits.dmi /:cl: Co-authored-by: nefarious <[email protected]> Co-authored-by: Drathek <[email protected]>
About the pull request
removes nazi jacket sprite from suits.dmi
Explain why it's good for the game
Testing Photographs and Procedure
Changelog
🆑
del: removes the nazi jacket sprite from suits.dmi
/:cl: