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

Expand cog1 public market, shrink stocks room #17522

Merged
merged 6 commits into from Jan 5, 2024

Conversation

Sovexe
Copy link
Contributor

@Sovexe Sovexe commented Jan 1, 2024

About the PR

  • Reduces the size of the cogmap1 stock exchange room
  • Increases the size of the public market stall and the info stall
  • Small location shuffle for bushes, cameras, lights, and ATMs to account for new layout
  • Removes stock exchange nuke target
  • Adds public market as a nuke target

Why's this needed?

  • Reallocate premium map real estate from one of the least utilized rooms into one of the most used for gimmicks and fun rooms

Changelog

(u)Sovexe
(+)The Cogmap1 public market has been expanded some, the stock exchange has been shrunk.

@boring-cyborg boring-cyborg bot added the A-Mapping A mapping change label Jan 1, 2024
@github-actions github-actions bot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Jan 1, 2024
@pgmzeta
Copy link
Contributor

pgmzeta commented Jan 1, 2024

Any concerns about it still being a nukie target at that size?

@Sbmhawk
Copy link
Contributor

Sbmhawk commented Jan 1, 2024

image
This seems way too small to still be a nukie target.

@Sovexe Sovexe merged commit c2521b9 into goonstation:master Jan 5, 2024
22 checks passed
@Sovexe Sovexe deleted the cog1-market branch January 5, 2024 15:37
github-actions bot pushed a commit that referenced this pull request Jan 5, 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 size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants