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

adds health variable to firelocks #7886

Merged
merged 1 commit into from Mar 22, 2022
Merged

adds health variable to firelocks #7886

merged 1 commit into from Mar 22, 2022

Conversation

ghost
Copy link

@ghost ghost commented Mar 19, 2022

About the PR

adds a health variable to firelocks so people can now smash them until they break

Why's this needed?

i think people being infinitely trapped by firelocks if they don't have a crowbar or are dealing with packetry is bad

(u)nefarious6th
(+)you can now break firelocks down by hitting them repeatedly, just like airlocks

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 19, 2022
@frawhst
Copy link
Member

frawhst commented Mar 20, 2022

This is cool and good in concept, will do some testing to see if the values are good

@ghost
Copy link
Author

ghost commented Mar 20, 2022

This is cool and good in concept, will do some testing to see if the values are good

thank you. i don't have the ability to test locally. was curious to know too if breaking will cause the same debris explosion as airlocks do since i didn't find a specific proc for it on airlock.dm

@TTerc TTerc merged commit 935c8b9 into goonstation:master Mar 22, 2022
github-actions bot pushed a commit that referenced this pull request Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants