Skip to content

Conversation

@out-of-phaze
Copy link
Member

@out-of-phaze out-of-phaze commented Dec 28, 2025

Description of changes

Split out of #5120.
Power debug objects are no longer a /obj/machinery/power subtype because, well, they don't need to be.

  • Map migration added

Why and what will this PR improve

Unnecessary subtyping is unnecessary.

Authorship

Me

@out-of-phaze out-of-phaze added the awaiting author This PR is awaiting action from the author before it can be merged. label Dec 28, 2025
@out-of-phaze out-of-phaze changed the title Make power debug objects not use the power subtype Make power debug machines not use the power subtype Dec 28, 2025
@out-of-phaze out-of-phaze marked this pull request as ready for review December 30, 2025 19:33
@out-of-phaze out-of-phaze force-pushed the codequality/debug-power branch from fbd71e3 to cbca446 Compare December 30, 2025 19:33
@out-of-phaze out-of-phaze added ready for review This PR is ready for review and merge. and removed awaiting author This PR is awaiting action from the author before it can be merged. labels Dec 30, 2025
@@ -0,0 +1,2 @@
/obj/machinery/power/debug_items/@SUBTYPES : /obj/machinery/debug_items/@SUBTYPES{@OLD}
/obj/machinery/power/ascent_reactor/@SUBTYPES : /obj/machinery/power/@SUBTYPES{@OLD} No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't seem correct.

Copy link
Member Author

Choose a reason for hiding this comment

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

oops

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.

Migration is using the wrong replacement path.

@out-of-phaze out-of-phaze force-pushed the codequality/debug-power branch from cbca446 to b4f6be0 Compare December 31, 2025 00:46
@MistakeNot4892 MistakeNot4892 merged commit 37a182c into NebulaSS13:dev Dec 31, 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