-
Notifications
You must be signed in to change notification settings - Fork 553
Ports Turf Fires #21659
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
base: master
Are you sure you want to change the base?
Ports Turf Fires #21659
Conversation
|
!review |
FabianK3
left a comment
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.
Very cool. With ZAS changes, subsystem and fire I would suggest a test-merge.
|
add attribution for turf_fire.dmi, looks to be taken from CM |
…intain a turf flame.
FabianK3
left a comment
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.
Code looks good to me, didn't find anything else. I'd still suggest we test-merge this one.
Nice work.
Adds turf fires (ported from Baystation, which ported them from Shiptest), which are now used instead of ZAS fires for fuel decals and flamethrowers. They also supplement ZAS fires in especially bad atmos fires.
This makes flamethrowers and related objects more suitable for AOE denial, rather than cooking the room (including yourself). They still do the latter, but less so than a gaseous fire.
To make flamethrowers actually useful against them, simple mobs now take brute damage when on fire.
Asset Licenses
The following assets that have not been created by myself are included in this PR: