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

.38 Special mindbreaker fix + ammo techfab #795

Merged
merged 6 commits into from
Feb 10, 2024

Conversation

FluffiestFloof
Copy link
Contributor

@FluffiestFloof FluffiestFloof commented Feb 8, 2024

About the PR

Does what the PR name says.
Fix the .38 Mindbreaker not actually injecting the mindbreaker.
Add the .38 cartridge and box to the ammo techfab, lethals only since that's what wizden did with it.
Add the .38 pratice to the security techfab.
Add the "Ammo" and "Weapons" category where appropriate.

Why / Balance

Uh

Technical details

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • tweak: .38 Special is now in the ammo techfab.
  • fix: Fixed .38 Special Mindbreaker not breaking minds.

@github-actions github-actions bot added S: Needs Review Changes: YML Changes any yml files labels Feb 8, 2024
VMSolidus
VMSolidus previously approved these changes Feb 10, 2024
@VMSolidus
Copy link
Contributor

Having finished testing this, I believe this to be ready to merge except for one small request. Do you think you could also add the ability to manufacture generic .38 special pistol magazines in Security Techfabs? There's as a general rule of thumb an ability to make generic magazines for each gun available to the crew at those, and while it can make speedloaders for the .38 revolver, it currently doesn't have an option to make spare magazines for .38 handguns.

I'm sure no one will use this for traitorous acts :trollface:
@VMSolidus VMSolidus merged commit 28cc34d into DeltaV-Station:master Feb 10, 2024
10 checks passed
DeltaV-Bot pushed a commit that referenced this pull request Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants