Skip to content

Fix a few ignored spacemandmm directives#9764

Merged
Detective-Google merged 1 commit intocmss13-devs:masterfrom
Drulikar:more_spaceman_silent_errors
Jun 26, 2025
Merged

Fix a few ignored spacemandmm directives#9764
Detective-Google merged 1 commit intocmss13-devs:masterfrom
Drulikar:more_spaceman_silent_errors

Conversation

@Drulikar
Copy link
Copy Markdown
Contributor

@Drulikar Drulikar commented Jun 20, 2025

About the pull request

This PR (in addition to #9762) fixes a few places where Spacemandmm directives like SHOULD_CALL_PARENT were doing nothing but also not emitting any warning. With SpaceManiac/SpacemanDMM#435 they will now be errors.

Explain why it's good for the game

Code should not merely be no-ops.

Changelog

No player facing changes.

@github-project-automation github-project-automation bot moved this to Awaiting Review in Review Backlog Jun 20, 2025
@cmss13-ci cmss13-ci bot added the Missing Changelog Maintainers always document their changes. label Jun 20, 2025
@github-project-automation github-project-automation bot moved this from Awaiting Review to Partially Approved / Mergeable in Review Backlog Jun 21, 2025
@Detective-Google Detective-Google added this pull request to the merge queue Jun 26, 2025
Merged via the queue into cmss13-devs:master with commit 8d6a5b9 Jun 26, 2025
26 checks passed
@github-project-automation github-project-automation bot moved this from Partially Approved / Mergeable to Merged in Review Backlog Jun 26, 2025
@Drulikar Drulikar deleted the more_spaceman_silent_errors branch June 26, 2025 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Missing Changelog Maintainers always document their changes.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants