Added more blood types to slaughter demons #992

Closed
QuillTheCreature wants to merge 0 commits from (deleted):entities/slaughterdemon/morebloodtypes into master
Member

Short description

I added (I think) most other common blood types used for the slaughter demons jaunt.

This refactors how the reagent group field is done, now allowing a reagent to be in multiple groups.
Therefore, the SD just needs to specify a reagent group now (Blood) instead of needing to list every blood reagent multiple times.

Why we need to add this

Bugfix.

Media (Video/Screenshots)

Checklist

Ingame Changelog

🆑 Quill

  • fix: Added most other blood types for the slaughter demon to jaunt on.
<!-- Link definitions, do not edit --> [guidelines]: https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html [ruleset]: https://rules.byrdstation.space/#h.vigrxrnffwji ## Short description I added (I think) most other common blood types used for the slaughter demons jaunt. This refactors how the reagent group field is done, now allowing a reagent to be in multiple groups. Therefore, the SD just needs to specify a reagent group now (Blood) instead of needing to list every blood reagent multiple times. ## Why we need to add this Bugfix. ## Media (Video/Screenshots) <video src="/attachments/5f19f674-56ca-4ccf-9e06-f6fcf91f580c" title="Screencast_20261004_135040" controls></video> ## Checklist <!-- Check boxes for faster reviewing of your PR --> - [x] I do not require assistance to complete the PR. - [x] I have verified that the changes work before posting/requesting review of my PR. - [x] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics. - [x] I have read the [Pull Request and Changelog Guidelines][guidelines] and [Byrd Station's Contributor Ruleset][ruleset]. ### Ingame Changelog :cl: Quill - fix: Added most other blood types for the slaughter demon to jaunt on.
Added more blood types to slaughter demons
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 6s
CRLF Check / CRLF Check (pull_request) Successful in 1m6s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 1m25s
RSI Validator / Validate RSIs (pull_request) Successful in 1m19s
Map file schema validator / YAML map schema validator (pull_request) Successful in 8m54s
YAML Linter / YAML Linter (pull_request) Successful in 8m55s
Test Packaging / Test Packaging (pull_request) Successful in 19m26s
cdc2cdcb1b
QuillTheCreature force-pushed entities/slaughterdemon/morebloodtypes from cdc2cdcb1b
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 6s
CRLF Check / CRLF Check (pull_request) Successful in 1m6s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 1m25s
RSI Validator / Validate RSIs (pull_request) Successful in 1m19s
Map file schema validator / YAML map schema validator (pull_request) Successful in 8m54s
YAML Linter / YAML Linter (pull_request) Successful in 8m55s
Test Packaging / Test Packaging (pull_request) Successful in 19m26s
to 9d0868f248
Some checks failed
Labels: Size / size-label (pull_request_target) Successful in 1m19s
Labels: PR / labeler (pull_request_target) Successful in 1m19s
CRLF Check / CRLF Check (pull_request) Successful in 2m14s
RSI Validator / Validate RSIs (pull_request) Successful in 1m20s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 1m44s
Map file schema validator / YAML map schema validator (pull_request) Successful in 7m58s
YAML Linter / YAML Linter (pull_request) Failing after 7m42s
Test Packaging / Test Packaging (pull_request) Successful in 17m23s
2026-10-04 19:07:09 +00:00
Compare
EchoOfNothing left a comment

Blood reagent doesn't have Blood group

`Blood` reagent doesn't have `Blood` group
It'd be great if blood would be in the blood group
Some checks failed
Labels: PR / labeler (pull_request_target) Failing after 25s
Labels: Size / size-label (pull_request_target) Successful in 20s
CRLF Check / CRLF Check (pull_request) Successful in 58s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 1m12s
RSI Validator / Validate RSIs (pull_request) Successful in 1m16s
Map file schema validator / YAML map schema validator (pull_request) Successful in 9m10s
YAML Linter / YAML Linter (pull_request) Failing after 10m21s
Test Packaging / Test Packaging (pull_request) Successful in 26m10s
55c96ee349
Author
Member

@EchoOfNothing wrote in #992 (comment):

Blood reagent doesn't have Blood group

Yeah you know, that'd be good.

@EchoOfNothing wrote in https://git.byrdstation.space/Byrd-Station/Byrd/pulls/992#issuecomment-8566: > `Blood` reagent doesn't have `Blood` group Yeah you know, that'd be good.
EchoOfNothing left a comment

LGTM. Thank you for your contribution!

LGTM. Thank you for your contribution!
Fixed yaml linter by converting all reagent group to arrays
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
PR Changelogs / changelog (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 4s
CRLF Check / CRLF Check (pull_request) Successful in 34s
Test Packaging / Test Packaging (pull_request) Successful in 13m13s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 41s
RSI Validator / Validate RSIs (pull_request) Successful in 45s
Map file schema validator / YAML map schema validator (pull_request) Successful in 5m15s
YAML Linter / YAML Linter (pull_request) Successful in 4m56s
d8026aa786
Author
Member

Fixed all yaml linter errors by writing a horrible piece of code.

For future references: Program

Fixed all yaml linter errors by writing a horrible piece of code. For future references: [Program](/attachments/453776f1-4517-473f-9948-06d22a642f41)
Owner

@QuillTheCreature
This pull request is broken due to missing fork information.

@QuillTheCreature `This pull request is broken due to missing fork information.`
QuillTheCreature closed this pull request 2026-10-08 19:58:06 +00:00
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
PR Changelogs / changelog (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Failing after 4s
CRLF Check / CRLF Check (pull_request) Successful in 34s
Test Packaging / Test Packaging (pull_request) Successful in 13m13s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 41s
RSI Validator / Validate RSIs (pull_request) Successful in 45s
Map file schema validator / YAML map schema validator (pull_request) Successful in 5m15s
YAML Linter / YAML Linter (pull_request) Successful in 4m56s

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Byrd-Station/Byrd!992
No description provided.