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

[PORT] Fix some bugs related to the xenomorph roundend report #2423

Merged
merged 2 commits into from
Jul 21, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Jun 26, 2024

About The Pull Request

Ports the following PRs:

Changelog

🆑
fix: (JohnFulpWillard) Science Xenos no longer turn the entire roundend report into bold letters.
fix: (Melbert) Captive Xeno end round report should make a tad more sense.
/:cl:

JohnFulpWillard and others added 2 commits June 27, 2024 14:47
## About The Pull Request

Fixes the 2% xeno's roundend report from turning the rest of the report
into bold letters.

## Why It's Good For The Game

look it's not all bold anymore

![image](https://github.com/tgstation/tgstation/assets/53777086/5b46dc02-5fb5-4d16-aedf-fc1c11aad045)

## Changelog

:cl:
fix: Science Xenos no longer turn the entire roundend report into bold
letters.
/:cl:
## About The Pull Request

`check_captivity` accepts a mob, not a mind


![image](https://github.com/tgstation/tgstation/assets/51863163/77952d56-8142-4bbc-8042-028dcb39a989)

Which leads me to believe this always returned `null` and always failed.

But checking further and this proc does not return a truthy or falsy
value at all, meaning even if this runtime didn't happen, it still
wouldn't function correctly. So I made it check for `CAPTIVE_XENO_PASS`.

## Changelog

:cl: Melbert
fix: Captive Xeno end round report should make a tad more sense
/:cl:
@dwasint dwasint merged commit 3eee741 into Monkestation:master Jul 21, 2024
20 checks passed
github-actions bot added a commit that referenced this pull request Jul 21, 2024
@Absolucy Absolucy deleted the boldnomore branch July 21, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants