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

give revs a chance #636

Merged
merged 1 commit into from
Jan 2, 2024
Merged

give revs a chance #636

merged 1 commit into from
Jan 2, 2024

Conversation

deltanedas
Copy link
Member

About the PR

increase headrevs from 1 max to 2 max with 60 people readied up, to give them a slight chance of winning

Why / Balance

i dont recall a single time revs took over the station, though its been close with certain good headrevs, being alone is crippling on highpop. since it requires a lot of people readying up it'll still usually be 1 headrev, though on 100p its quite possible to have 2 now.

ideally theyd be told who other headrevs are roundstart to coordinate with them instead of having the round ruined by someone flashing in hallways 5 seconds into the round and instantly alerting everyone, but thats for upstream

Technical details

no

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

no

Changelog
🆑

  • tweak: On very high pop revolution rounds, there can be 2 head revolutionaries.

Copy link
Contributor

@Colin-Tel Colin-Tel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like an alright idea to me based on the majority of Rev rounds being unsuccessful at a higher population. Let's try it out.

@Colin-Tel Colin-Tel merged commit 443d516 into DeltaV-Station:master Jan 2, 2024
12 checks passed
DeltaV-Bot pushed a commit that referenced this pull request Jan 2, 2024
DebugOk pushed a commit to DebugOk/Delta-v that referenced this pull request Jan 20, 2024
Co-authored-by: deltanedas <@deltanedas:kde.org>
(cherry picked from commit 443d516)
DebugOk pushed a commit to DebugOk/Delta-v that referenced this pull request Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants