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] Bit of reference handling for custom food holders #1789

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#728
Original PR: tgstation/tgstation#81195

About The Pull Request

ingredients seems to hold a reference to all the food in our atom's contents that are used to make up our custom food, but it doesn't clear it anywhere.

So I implemented Exited, as well as cutting the list on component Destroy.

* Bit of reference handling for custom food holders (#81195)

## About The Pull Request

`ingredients` seems to hold a reference to all the food in our atom's
contents that are used to make up our custom food, but it doesn't clear
it anywhere.

So I implemented `Exited`, as well as cutting the list on component
`Destroy`.

* Bit of reference handling for custom food holders

---------

Co-authored-by: MrMelbert <[email protected]>
@Iajret Iajret merged commit a44203e into master Feb 6, 2024
23 checks passed
@Iajret Iajret deleted the upstream-mirror-728 branch February 6, 2024 11:24
Iajret pushed a commit that referenced this pull request Apr 3, 2024
…ion (#81770) (#1789)

Bumps up several GitHub `actions` to the Node 20 Version (#81770)

Node 16 actions are deprecated, we need to use Node 20 actions now (as
noted by this handy screenshot which spams it on every CI run)

![image](https://github.com/tgstation/tgstation/assets/34697715/24ea3013-c762-4027-951c-d598b1eec8a3)

You may see
https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/
for more information.

Co-authored-by: san7890 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants