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

Move the supermatter crystal and related content into a modpack #4691

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

out-of-phaze
Copy link
Member

Description of changes

Moves the supermatter crystal and related content into a modpack.

Depends on #4690.

I have no clue how I'd test this. Engine setup verb, check that the UIs work, fix atmos grief verb, try a delam?

Why and what will this PR improve

Moves the SM into a modpack, which is a good proof-of-concept for "what if we modpacked base-game SS13 features?" Like, I know that Shaded Hills doesn't suffer at all from "supermatter crystals existing in the game code but being inaccessible," but modpacking it highlights places our modularity is lacking, and it makes it easier for codebases to pick-and-choose what they want in their lore; some might not want supermatter crystals to be orderable at all, for example.

Authorship

Me.

Changelog

Shouldn't be any user-facing changes?

@out-of-phaze out-of-phaze added do not merge This PR is on hold, do not merge it. has dependencies This PR should not be merged prior to any PRs linked in body or comments. work in progress This PR is under development and shouldn't be merged. labels Jan 3, 2025
@out-of-phaze out-of-phaze force-pushed the modular/supermatter branch 4 times, most recently from eee801a to 825f4de Compare January 3, 2025 03:59
MistakeNot4892
MistakeNot4892 previously approved these changes Jan 7, 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.

This all seems conceptually fine to me. Have to admit I did not read the copied/moved code in great depth.

@out-of-phaze out-of-phaze dismissed MistakeNot4892’s stale review January 9, 2025 00:06

The merge-base changed after approval.

@MistakeNot4892 MistakeNot4892 added needs rebase/retarget This PR is targetting the wrong branch or has included commits from another branch. and removed has dependencies This PR should not be merged prior to any PRs linked in body or comments. labels Jan 9, 2025
@MistakeNot4892 MistakeNot4892 removed the needs rebase/retarget This PR is targetting the wrong branch or has included commits from another branch. label Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge This PR is on hold, do not merge it. work in progress This PR is under development and shouldn't be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants