Skip to content

June 2026 Dead Code Cleanup#12445

Open
fira wants to merge 16 commits into
cmss13-devs:masterfrom
fira:fira-codetrim-2026
Open

June 2026 Dead Code Cleanup#12445
fira wants to merge 16 commits into
cmss13-devs:masterfrom
fira:fira-codetrim-2026

Conversation

@fira

@fira fira commented Jun 14, 2026

Copy link
Copy Markdown
Member

About the pull request

This just removes dead code from the game, that is presumably not usable at all in the game builds.

Some highlights:

  • Removes Crash/Dropship logic from old Shuttle backend - unfortunately it can't be completely removed because the ASRS Supply elevator still uses old shuttles
  • Removes DOREC specific code that hasn't been used for a decade
  • Removes orphaned admin tools backend that weren't actually usable
  • Removes species handlers that weren't actually used at all
  • Removes speech change pathogen (and the associated templated bottle) that actually didn't do anything because nothing cares about special_voice

Explain why it's good for the game

Especially removal of duplicated explosion and shuttle code make it easier for newer contributors to pick things up and navigate the codebase, without getting also misled by antique code that is out of standards.

Changelog

🆑
code: Removed unused game code.
/:cl:

@github-project-automation github-project-automation Bot moved this to Awaiting Review in Review Backlog Jun 14, 2026
@cmss13-ci cmss13-ci Bot added Code Improvement Make the code longer size/XL Denotes a PR that changes 799-1999 lines, ignoring generated files. labels Jun 14, 2026
@fira fira changed the title June 26 Dead Code Cleanup June 2026 Dead Code Cleanup Jun 14, 2026
@fira

fira commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

Get #12412 first

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jun 16, 2026
@cm13-github

Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cmss13-ci

cmss13-ci Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@cmss13-ci cmss13-ci Bot added the Stale beg a maintainer to review your PR label Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Improvement Make the code longer Merge Conflict PR can't be merged because it touched too much code size/XL Denotes a PR that changes 799-1999 lines, ignoring generated files. Stale beg a maintainer to review your PR

Projects

Status: Awaiting Review

Development

Successfully merging this pull request may close these issues.

2 participants