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] Fixes create and destory failure due to missing physiology #633

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

MysticalFaceLesS
Copy link
Member

Оригинальный PR: shiptest-ss13#2999

About The Pull Request

Sometimes during unit tests (prob shouldn't happen in the first place) a human will get created and have electrocute act called by a tesla anomaly which causes a fail because it looks for a shock resist in physiology which doesn't yet exist.
If humans are shocked during init and it does not have its physiology yet it now returns
It may be better to just have an if physiology for fetching the physiology siemens but if its missing its physiology its prob missing other thing needed like generating organs or dna

Why It's Good For The Game

im tired of half my prs failing from this

Changelog

🆑FalloutFalcon
fix: fixed create and destroy test fail
/:cl:

@github-actions github-actions bot added 🪓 НЕ МОДУЛЬНО 🪓 Не мудульно да, переделать бы 🧱 Код 🧱 Изменения .dm файлов 🔵 Требуется проверка labels Jun 2, 2024
@MrCat15352 MrCat15352 merged commit 0c648bc into beta-dev Jun 2, 2024
3 checks passed
@MrCat15352 MrCat15352 deleted the upstream-pr-2999 branch June 2, 2024 12:10
@MrCat15352 MrCat15352 added ☠Слияние с ОФАМИ☠ PR говорит сам за себя. Подтягиваем последние обновы офов... Да хранит нас дядя Целадонский and removed 🪓 НЕ МОДУЛЬНО 🪓 Не мудульно да, переделать бы labels Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☠Слияние с ОФАМИ☠ PR говорит сам за себя. Подтягиваем последние обновы офов... Да хранит нас дядя Целадонский 🔵 Требуется проверка 🧱 Код 🧱 Изменения .dm файлов
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants