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 Clarion's central maintenance apc #17841

Merged

Conversation

TheTyrant25
Copy link
Contributor

[MAPPING] [BUG]

About the PR

Sequel to #17827.

Moves and renames the central maint APC. Previously it was behind a heads only door and confusingly called armoury APC (it didn't control the armoury).

Why's this needed?

Bugs bad. There shouldn't be two armoury APCs, that's misleading. And central maintenance's APC shouldn't be behind a head access door, it just hassles engineers trying to fix/sabotage it.

Changelog

(u)Tyrant
(+)Clarion's central maintenance APC has been moved to actually be in central maintenance (and is no longer confusingly called armoury APC)

@keywordlabeler keywordlabeler bot added A-Mapping A mapping change C-Bug A bug that impacts usage of a feature labels Feb 2, 2024
@boring-cyborg boring-cyborg bot added the A-Mapping A mapping change label Feb 2, 2024
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 2, 2024
@TobleroneSwordfish TobleroneSwordfish merged commit 9ccdf45 into goonstation:master Feb 2, 2024
25 checks passed
@TheTyrant25 TheTyrant25 deleted the clarion-apc-nonsense branch February 2, 2024 17:24
github-actions bot pushed a commit that referenced this pull request Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Mapping A mapping change C-Bug A bug that impacts usage of a feature size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants