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

Deployable turrets eject bullet casings #18987

Merged
merged 2 commits into from
May 26, 2024

Conversation

FlameArrow57
Copy link
Contributor

@FlameArrow57 FlameArrow57 commented May 18, 2024

[GAME OBJECTS][FEATURE][REWORK]

About the PR

This PR makes it so that deployable turrets (N.A.R.C.S./NARCS and NAS-T) eject bullet casings. They now have a forensics profile too, to match with their casings

Also does a minor rework of variables/procs internally to make it work

image

image

Why's this needed?

Thought it would be a fun effect

Changelog

(u)FlameArrow57
(+)Deployable turrets like N.A.R.C.S. turrets now eject bullet casings and have their own forensics profile.

@keywordlabeler keywordlabeler bot added C-Feature A new feature or enhancements to existing features C-Rework Reworks a feature labels May 18, 2024
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label May 18, 2024
@keywordlabeler keywordlabeler bot added the A-Game-Objects The point of this PR is to deal with a specific game object label May 18, 2024
@TobleroneSwordfish
Copy link
Contributor

These things do have infinite ammo so I kinda worry about generating an infinite pile of casings if the turret gets stuck shooting a cockroach or something. Maybe start deleting them when they go over a certain amount on one tile?

Copy link
Member

@Tarmunora Tarmunora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks good, some solution to the infinite-stack-of-casings issue would be good though yeah

@FlameArrow57
Copy link
Contributor Author

I added something simple to make it start deleting casings once too many turf contents have been reached, LMK if something more involved is wanted

@github-actions github-actions bot added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label May 26, 2024
@Tarmunora Tarmunora merged commit d5cf00b into goonstation:master May 26, 2024
23 checks passed
github-actions bot pushed a commit that referenced this pull request May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Game-Objects The point of this PR is to deal with a specific game object C-Feature A new feature or enhancements to existing features C-Rework Reworks a feature S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants