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 runtime error when destroying and building gauss turrets #2609

Merged
merged 1 commit into from
Mar 29, 2024

Conversation

Bobbanz1
Copy link
Contributor

About The Pull Request

This PR fixes a runtime error I encountered while trying to recreate another bug that was reported a few weeks back.

The runtime error in question was this:

Skärmbild 2024-02-29 101248

I fixed it by giving the machine board for the gauss turret an empty req_components list.
Side effect of fixing it seemed to make the turret itself now leave behind a wired machine frame alongside the board instead of just the board.

Why It's Good For The Game

Makes it less resource consuming to rebuild a destroyed gauss turret since it now leaves behind the machine frame that had been used to build it.
Makes the construction and destruction of the gauss turret not produce a runtime error.

Testing Photographs and Procedure

Screenshots&Videos

Go hit a gauss turret until it's destroyed.

Skärmbild 2024-02-29 110751

Changelog

🆑
fix: The destruction of a gauss turret now leaves behind the machine frame that had been used to construct it.
code: Gave the machine board of the gauss turret a req_components list with nothing inside it to prevent a runtime error from occurring.
/:cl:

Copy link
Contributor

@Bokkiewokkie Bokkiewokkie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moment

@Bokkiewokkie Bokkiewokkie merged commit 202cd8b into BeeStation:master Mar 29, 2024
9 checks passed
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.

2 participants