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

Refactors lava into being a component with different types for toxic water and plasma rivers #22017

Merged
merged 5 commits into from
May 28, 2024

Conversation

Moltijoe
Copy link
Contributor

@Moltijoe Moltijoe commented May 6, 2024

it was initially just copying code from lava over to toxic water
but i decided it would be better if refactored into a component
so now it is

Why is this good for the game?

code quality + consistency between similar features
also, it makes it easier to create new types of terrain that do stuff to people standing on it
(could make healing terrain that heals instead of doing damage)

Testing

image
image

🆑
tweak: Refactors lava into being a component with different types for toxic water and plasma rivers
bugfix: can now rcd toxic water
bugfix: boats and grilles now protect from toxic water
experimental: please tell me DIRECTLY if there's anything wonky about lava/plasma/toxic water
/:cl:

@Yogbot-13 Yogbot-13 added Fix This fixes an issue. Please link issues in fix PRs Tweak This PR contains changes to the game. labels May 6, 2024
@Moltijoe Moltijoe added the Awaiting - Action - Testing TEST YOUR STUFF label May 6, 2024
@Moltijoe Moltijoe marked this pull request as draft May 26, 2024 00:36
@Moltijoe Moltijoe changed the title Modifies toxic water on jungleland to function more like lava Refactors lava into being a component with different types for toxic water and plasma rivers May 26, 2024
@Moltijoe Moltijoe removed the Awaiting - Action - Testing TEST YOUR STUFF label May 26, 2024
@Moltijoe Moltijoe marked this pull request as ready for review May 26, 2024 01:41
@SapphicOverload SapphicOverload merged commit 5fe1430 into yogstation13:master May 28, 2024
14 checks passed
@Yogbot-13 Yogbot-13 added the DME Edit This PR affects the yogstation.DME file label May 28, 2024
Yogbot-13 added a commit that referenced this pull request May 28, 2024
@Moltijoe Moltijoe deleted the JungleWaterProtection branch May 28, 2024 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DME Edit This PR affects the yogstation.DME file Fix This fixes an issue. Please link issues in fix PRs Tweak This PR contains changes to the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants