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

Bomb test chamber for donut 3 #16114

Merged
merged 7 commits into from Nov 15, 2023
Merged

Conversation

CheffieGithub
Copy link
Contributor

@CheffieGithub CheffieGithub commented Sep 29, 2023

About the PR

Adds an explosive mass driver area and bomb test chamber on donut 3.
Adds auto television subtype.

The chamber itself is themed as a playing stage with some themed items, connected to the station by lattices to allow for crew to access it by walking across them. Bombs will hit the stage and land on the big X in the centre of the structure.

Diffbot render:
68747470733a2f2f616666656374656461726330372e626c6f622e636f72652e77696e646f77732e6e65742f6d6462322f696d616765732f3235323034373438332f31373932383037313635322f6d2f6d6170735f646f6e7574332f302d61667465722e706e67

Lattices are all auto turf connecting.

Known issue is that the auto televisions are all called security camera due to the bad old auto name code. Will fix this in a later PR.

Why's this needed?

Bomb chambers are convient for testing or producing pressure crystals

Changelog

(u)Cheffie
(*)Donut 3 has been outfitted with a testing chamber for toxins.

@boring-cyborg boring-cyborg bot added the A-Mapping A mapping change label Sep 29, 2023
@github-actions github-actions bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Sep 29, 2023
@CheffieGithub CheffieGithub marked this pull request as ready for review September 29, 2023 22:57
@CheffieGithub CheffieGithub added the C-Feature A new feature or enhancements to existing features label Sep 29, 2023
@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 19, 2023
@pali6 pali6 removed the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 19, 2023
@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Oct 21, 2023
@github-actions github-actions bot removed the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Oct 21, 2023
@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Oct 30, 2023
@github-actions github-actions bot removed the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Nov 2, 2023
@pali6 pali6 merged commit c181f5f into goonstation:master Nov 15, 2023
22 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 15, 2023
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-Feature A new feature or enhancements to existing features size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants