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

Fully Revert Clown Waddling #32652

Merged
merged 1 commit into from
Oct 6, 2024

Conversation

Golinth
Copy link
Contributor

@Golinth Golinth commented Oct 5, 2024

About the PR

Fully reverts waddling, revival of #29161

Why / Balance

  • Requested by Sloth here and further approved of here. A couple other people in the discord, including a game admin, also requested this.

  • The original PR was closed by the creator deleting their head repository, not by any consensus.

Waddling as implemented is entirely broken, the original creator is no longer working on SS14, and it needs reimplementing according to Sloth's reviews (something entirely above my skill level). I am sad to see it go, but as-is it makes the game look and feel worse as a clown.

Technical details

Reverts #26338, #26965, #26983, and #27628

Media

clown-neutral-face-white-46962104-2392398420

Requirements

Breaking Changes

Removed:
WaddleAnimationSystem
WaddleWhenWornComponent
WaddleClothingSystem
WaddleAnimationComponent
SharedWaddleAnimationSystem

Changelog

🆑

  • remove: Removed clown waddling until implemented properly

@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged. label Oct 5, 2024
@regenpower
Copy link

regenpower commented Oct 5, 2024

1984, but all the clown replays are broken so it makes sense yeah

@slarticodefast
Copy link
Member

slarticodefast commented Oct 5, 2024

Yeah, this is necessary, waddling has too many bugs.
Could you mention the removed components and systems in the breaking changes section of your PR in case forks use them for something else?

@deltanedas
Copy link
Contributor

as said the original author will not work on this, so waddling will be gone for at least a year before anyone cares enough to rework it all

@Golinth
Copy link
Contributor Author

Golinth commented Oct 6, 2024

Yeah, this is necessary, waddling has too many bugs. Could you mention the removed components and systems in the breaking changes section of your PR in case forks use them for something else?

should be good now

@slarticodefast slarticodefast added PR: Approved Indicates that a PR has been approved by a maintainer and it may be merged soon by them. Undergoing Maintainer Discussion This PR is currently going through the 72-hour discussion window as per maintainer policy and removed Status: Needs Review This PR requires new reviews before it can be merged. labels Oct 6, 2024
@slarticodefast slarticodefast self-assigned this Oct 6, 2024
@slarticodefast
Copy link
Member

In case someone wants to work on a proper reimplementation, some of the requirements are mentioned here: #29156

@slarticodefast slarticodefast merged commit 46a2eb5 into space-wizards:master Oct 6, 2024
13 checks passed
Ilya246 pushed a commit to Ilya246/space-station-14 that referenced this pull request Oct 7, 2024
Fully revert Clown Waddling (revival of space-wizards#29161)

A sad day, see space-wizards#29156 for discussion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Approved Indicates that a PR has been approved by a maintainer and it may be merged soon by them. Undergoing Maintainer Discussion This PR is currently going through the 72-hour discussion window as per maintainer policy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants