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

Removes LAssailant #11354

Merged
merged 2 commits into from
Aug 23, 2024
Merged

Conversation

PowerfulBacon
Copy link
Member

@PowerfulBacon PowerfulBacon commented Aug 21, 2024

About The Pull Request

Removes LAssailant. I was going to remove it but this logic on slimes makes literally no sense. If someone attacks the silmes food the slime becomes friends with them? How does that make any sense.

Why It's Good For The Game

Cleans up the code.

Testing Photographs and Procedure

image

Compiles and runs. See code.

Changelog

🆑
code: Cleans up the code by removing LAssailant from mobs. Attacking something that a slime is feeding on will no longer cause it to become your friend.
/:cl:

@PowerfulBacon PowerfulBacon marked this pull request as ready for review August 21, 2024 19:18
@Tsar-Salat
Copy link
Contributor

code: Cleans up the code by removing LAssailant from mobs

Player facing changes?

Copy link
Member

@EvilDragonfiend EvilDragonfiend left a comment

Choose a reason for hiding this comment

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

LAssailant is only used for slime codes. We have no reason to keep this when it makes codes annoying to read.

@Rukofamicom
Copy link
Contributor

This will prevent slimes from becoming friendly to Xenobio won't it?

Slime "friends" can issue commands to slimes such as ordering them to attack and it's a really cool, if underutilized, mechanic. I've seen it used against halo cultists to flip what otherwise was a landslide cultist victory in a really funny way.

@PowerfulBacon
Copy link
Member Author

PowerfulBacon commented Aug 22, 2024

This will prevent slimes from becoming friendly to Xenobio won't it?

Slime "friends" can issue commands to slimes such as ordering them to attack and it's a really cool, if underutilized, mechanic. I've seen it used against halo cultists to flip what otherwise was a landslide cultist victory in a really funny way.

The logic for someone becoming a friend is used elsewhere, although it seems just as bizarre and unintuitive in other places. The code quality for slimes is absolutely dreadful and seems to have been written when space station 13 was in its infancy.

Edit: There were 2 ways that you previous could become friends with a slime: Feeding it plasma and attacking it's food. The second one makes absolutely no sense, so now you feed them plasma to become friends with them.

@Tsar-Salat
Copy link
Contributor

The logic for someone becoming a friend is used elsewhere, although it seems just as bizarre and unintuitive in other places. The code quality for slimes is absolutely dreadful and seems to have been written when space station 13 was in its infancy.

tgstation/tgstation#82176

This should probably be a long-term goal

@Rukofamicom Rukofamicom added this pull request to the merge queue Aug 23, 2024
Merged via the queue into BeeStation:master with commit 05cd61f Aug 23, 2024
8 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.

4 participants