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 S: Needs Review Status: Requires additional reviews before being fully accepted 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 S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. S: Undergoing Maintainer Discussion Status: Currently going through an extended discussion amongst maintainers, as per procedure. and removed S: Needs Review Status: Requires additional reviews before being fully accepted 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
@Golinth Golinth deleted the Revert-Waddling branch November 19, 2024 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. S: Undergoing Maintainer Discussion Status: Currently going through an extended discussion amongst maintainers, as per procedure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants