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

Minimum player count for nuclear emergency #17573

Conversation

flappybatpal
Copy link
Contributor

About the PR

Adds a 15 minimum ready player count to the nuclear mode.
Adds a build define to skip game mode checks, replaces THE_REVOLUTION_WILL_NOT_BE_TELEVISED.

Why's this needed?

Low player counts isn't ideal for team modes.
Nuclear can start with just two players right now.
15 players suggested by Gannets.

Changelog

(u)Flappybat
(*)The Nuclear Emergency game mode now requires 15 players.

@flappybatpal flappybatpal added C-Balance Balance changes, buffs and nerfs A-Gamemodes Changes or additions to any gamemode's main features labels Jan 5, 2024
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 5, 2024
_std/__build.dm Outdated Show resolved Hide resolved
code/datums/gamemodes/nuclear.dm Outdated Show resolved Hide resolved
code/datums/gamemodes/nuclear.dm Outdated Show resolved Hide resolved
@github-actions github-actions bot added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label Jan 6, 2024
@ZeWaka ZeWaka merged commit 796187f into goonstation:master Jan 7, 2024
23 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Gamemodes Changes or additions to any gamemode's main features C-Balance Balance changes, buffs and nerfs S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) 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

4 participants