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

[MIRROR] Romerol tumors in living non-zombies don't delete on death #845

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25278
Original PR: tgstation/tgstation#79945

About The Pull Request

Romerol tumors have been broken for over a month because Melbert made them delete on host death, without realizing that people without TRAIT_NODEATH can have them (namely, non-zombies). This simply puts the self-deletion behind an iszombie check.

Why It's Good For The Game

This was an oversight that went under the radar until I saw someone get denied a zombie outbreak because of it.

Changelog

🆑 Y0SH1M4S73R
fix: People exposed to romerol while alive will once again revive as zombies on death.
/:cl:

…MDB IGNORE] (#25278)

* Romerol tumors in living non-zombies don't delete on death (#79945)

## About The Pull Request

Romerol tumors have been broken for over a month because Melbert made
them delete on host death, without realizing that people without
`TRAIT_NODEATH` can have them (namely, non-zombies). This simply puts
the self-deletion behind an `iszombie` check.

## Why It's Good For The Game

This was an oversight that went under the radar until I saw someone get
denied a zombie outbreak because of it.

## Changelog

:cl:
fix: People exposed to romerol while alive will once again revive as
zombies on death.
/:cl:

Co-authored-by: Mothblocks <35135081+Mothblocks@ users.noreply.github.com>

* Romerol tumors in living non-zombies don't delete on death

---------

Co-authored-by: Y0SH1M4S73R <[email protected]>
Co-authored-by: Mothblocks <35135081+Mothblocks@ users.noreply.github.com>
@ReezeBL ReezeBL merged commit 824ae6c into master Nov 28, 2023
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-25278 branch November 28, 2023 05:17
AnywayFarus added a commit that referenced this pull request Nov 28, 2023
Iajret pushed a commit that referenced this pull request Feb 8, 2024
* Adds head-only target hotkey (#81222)

## About The Pull Request

Adds a new optional hotkey that targets only the head and does not cycle
to eyes or mouth.

Currently the way the numpad hotkeys work on /tg/ is every key targets
one body part... except 8. Numpad 8 Currently cycles through
head-eyes-mouth, despite _**numpad 7 already handling mouth and numpad 9
already handling eyes**_. There's no head-only key.

* Adds head-only target hotkey

---------

Co-authored-by: Swift <[email protected]>
ReezeBL pushed a commit that referenced this pull request Feb 9, 2024
* Adds head-only target hotkey (#81222)

## About The Pull Request

Adds a new optional hotkey that targets only the head and does not cycle
to eyes or mouth.

Currently the way the numpad hotkeys work on /tg/ is every key targets
one body part... except 8. Numpad 8 Currently cycles through
head-eyes-mouth, despite _**numpad 7 already handling mouth and numpad 9
already handling eyes**_. There's no head-only key.

* Adds head-only target hotkey

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: Swift <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants