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

Ports Fixes to some Looping Systems, LightBehaviorSystem Refactor From Wizden #1552

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

spess-empyrean
Copy link

Description

Ports over these commits from Wizden:
space-wizards/space-station-14@32e315c
space-wizards/space-station-14@8d015f5
Namely the first is a general refactor to LightBehaviorSystem and the second is a bugfix that fixes some looping behaviors like the low battery effect on a flashlight never stopping. If it's not atomic enough, let me know and I can separate them into two.

Changelog

🆑

  • fix: Fixed some errors with looping behaviors like jitter.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: YML Changes any yml files labels Jan 15, 2025
@SimpleStation14 SimpleStation14 changed the title Ports fixes to some looping systems, LightBehaviorSystem refactor from Wizden Ports Fixes to some Looping Systems, LightBehaviorSystem Refactor From Wizden Jan 15, 2025
Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

There's more below what I review requested, but cannot request changes to "Remove Yanderedev" on lines you aren't touching in this PR.

Content.Client/Light/EntitySystems/LightBehaviorSystem.cs Outdated Show resolved Hide resolved
Content.Client/Light/EntitySystems/LightBehaviorSystem.cs Outdated Show resolved Hide resolved
Co-authored-by: VMSolidus <[email protected]>
Signed-off-by: spess-empyrean <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants