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] Poly speech/memory fix. #750

Merged
merged 2 commits into from
Nov 21, 2023
Merged

[MIRROR] Poly speech/memory fix. #750

merged 2 commits into from
Nov 21, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25159
Original PR: tgstation/tgstation#79851

About The Pull Request

Simple fix for Poly's AI, Read Memory was returning null into a list used for Poly's speech causing a stack error. I changed the returned value to the empty returnable list to negate this problem.

Fixes #79844
Fixes #79859
Fixes #79865

First time making a pull request on Github, please bear with me if i make any mistakes.

Why It's Good For The Game

Less errors on the servers side. Probably saving the headache of debugging from some other poor soul.

* Poly speech/memory fix. (#79851)

Simple fix for Poly's AI, Read Memory was returning null into a list
used for Poly's speech causing a stack error. I changed the returned
value to the empty returnable list to negate this problem.

Fixes #79844
Fixes #79859
Fixes #79865

First time making a pull request on Github, please bear with me if i
make any mistakes.

Less errors on the servers side. Probably saving the headache of
debugging from some other poor soul.

---------

Co-authored-by: san7890 <the@ san7890.com>

* Poly speech/memory fix.

---------

Co-authored-by: shwb1 <[email protected]>
Co-authored-by: san7890 <the@ san7890.com>
@Steals-The-PRs Steals-The-PRs added Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror labels Nov 21, 2023
@Iajret Iajret merged commit 266cdfc into master Nov 21, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-25159 branch November 21, 2023 23:40
Iajret pushed a commit that referenced this pull request Feb 3, 2024
* [Testing] Fixing cleanbot AI related CI runtimes

* Update find_and_set.dm

* Why does this still keep happening?!!?!?

* Update cleanbot_ai.dm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants