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

Fixes Static Anomalies #2897

Merged
merged 3 commits into from
Apr 19, 2024
Merged

Fixes Static Anomalies #2897

merged 3 commits into from
Apr 19, 2024

Conversation

Sun-Soaked
Copy link
Member

@Sun-Soaked Sun-Soaked commented Apr 14, 2024

About The Pull Request

bzzt. kss bzzt kzz (fixes the static anomaly, it no longer does a weird runtime related to attempting to run mob code on a spawner effect. The resulting fucked up chat message is also fixed.)

bzzzap kzzazz mmm bzzzaap (Adds a weakref on mob spawners pointing to their spawned mob. This is also a surprise tool that'll help us later w. stuff related to manipulating entities after they've been spawned. I'll prolly port the full mob spawner rework at some point)

Why It's Good For The Game

bzzz

Changelog

🆑
fix: planetary static anomalies no longer drop a broken spawner.
code: mob spawners now store a weakref on their spawned object
/:cl:

@github-actions github-actions bot added the Code change Watch something violently break. label Apr 14, 2024
@thgvr thgvr added this pull request to the merge queue Apr 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 16, 2024
@thgvr thgvr added this pull request to the merge queue Apr 19, 2024
Merged via the queue into shiptest-ss13:master with commit 9e9b5ff Apr 19, 2024
14 checks passed
MysticalFaceLesS pushed a commit to CeladonSS13/Shiptest that referenced this pull request Apr 20, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request
bzzt. kss bzzt kzz (fixes the static anomaly, it no longer does a weird
runtime related to attempting to run mob code on a spawner effect. The
resulting fucked up chat message is also fixed.)

bzzzap kzzazz mmm bzzzaap (Adds a weakref on mob spawners pointing to
their spawned mob. This is also a surprise tool that'll help us later w.
stuff related to manipulating entities after they've been spawned. I'll
prolly port the full mob spawner rework at some point)
<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->

## Why It's Good For The Game
bzzz
<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

## Changelog

:cl:
fix: planetary static anomalies no longer drop a broken spawner.
code: mob spawners now store a weakref on their spawned object
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants