Skip to content

Conversation

@out-of-phaze
Copy link
Member

Description of changes

Split out of #5120.
The first commit improves the gravity generator's code quality.
The second commit removes the gravity generator.
This is supposedly done in two commits so that if someone wants to readd the gravity generator they can just revert the second one and get a version with better code quality, but in reality I just cleaned up the code before realizing it wasn't used and didn't want to waste my effort, so...

Why and what will this PR improve

Removes some unused, really janky code.

Authorship

Me

@out-of-phaze out-of-phaze added the ready for review This PR is ready for review and merge. label Dec 28, 2025
MistakeNot4892
MistakeNot4892 previously approved these changes Dec 29, 2025
Copy link
Contributor

@MistakeNot4892 MistakeNot4892 left a comment

Choose a reason for hiding this comment

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

I didn't even know we HAD a gravity generator. If this functionality is desirable it should be added to the intertial damper.

@MistakeNot4892 MistakeNot4892 added the needs rebase/retarget This PR is targetting the wrong branch or has included commits from another branch. label Dec 29, 2025
Copy link
Contributor

@MistakeNot4892 MistakeNot4892 left a comment

Choose a reason for hiding this comment

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

Conflict resolution

@MistakeNot4892 MistakeNot4892 removed the needs rebase/retarget This PR is targetting the wrong branch or has included commits from another branch. label Dec 30, 2025
@MistakeNot4892 MistakeNot4892 merged commit 637d181 into NebulaSS13:dev Dec 30, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review This PR is ready for review and merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants