Skip to content

Military & Sec Belt subtypes can now hold One Medium-Sized Pistol #5052

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

Merged
merged 3 commits into from
Jul 13, 2025

Conversation

generalthrax
Copy link
Contributor

@generalthrax generalthrax commented Jul 9, 2025

About The Pull Request

thanks to erika and zym for helping me out here

All belt/military and belt/security can now fit one medium-sized pistol (along with a few energy weapons of similar size)

fixes a few storage things to make it possible

Why It's Good For The Game

The sprites visually have what I think is a holster and I think a belt holster is probably good

Changelog

🆑
balance: Military and Security belts can now fit one Medium-Sized Pistol (And Other Small Energy Weapons)
fix: Fixed storage's exception_hold so some items can actually be exempted from a size restriction
/:cl:

@generalthrax generalthrax requested a review from a team as a code owner July 9, 2025 04:19
@github-actions github-actions bot added the Code change Watch something violently break. label Jul 9, 2025
@generalthrax generalthrax changed the title Military & Sec Belts can now hold One Medium-Sized Pistol Military & Sec Belt subtypes can now hold One Medium-Sized Pistol Jul 9, 2025
@MassiveMen
Copy link
Contributor

Its so peak.......

@Aquidu
Copy link
Contributor

Aquidu commented Jul 10, 2025

i'm not exactly against this, but i do wonder if this would make shoulder holsters obsolete? in addition, if you use one in conjunction with a combat belt you could hold up to three guns at once, and that isn't even including one for your back/in your bag

@generalthrax
Copy link
Contributor Author

generalthrax commented Jul 10, 2025

They're pistols and you're compromising extra storage on your belt. Were assault / tool / medical belts made obsolete by the addition of utility / ammo / medical pouches? It's just a choice you make if you wanna pack extra firepower in exchange for giving up the option of carrying something else.

@Imaginos16 Imaginos16 added this pull request to the merge queue Jul 13, 2025
Merged via the queue into shiptest-ss13:master with commit bc846a5 Jul 13, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants