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

Limit "Die a glorious death" objective to one traitor per round. #1819

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

beck-thompson
Copy link
Contributor

About the PR

See the title!

Why / Balance

A lot of shifts (Especially with a lot of traitors) can end up having three or four people with the objective which ends up very chaotic and not fun. Limiting it to one per round still allows the objective to come up, just not dominate the round!

Changelog

🆑 Beck Thompson

  • tweak: The "Die a glorious death" objective is now limited to one traitor per round maximum!

@deltanedas

This comment was marked as outdated.

@beck-thompson
Copy link
Contributor Author

i think that means 2 max

When I tested it only let one person have the objective so I think its OK? Looking at the code, it also appears to make sense and should work like I thought. Also your the one who wrote it so if it doesn't work I blame you 🧌

@beck-thompson
Copy link
Contributor Author

escape characters do not appear to work on github so that emoji is not correct 😭

@deltanedas
Copy link
Member

:trollface: :trollface:

Copy link
Member

@deltanedas deltanedas left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@TadJohnson00 TadJohnson00 left a comment

Choose a reason for hiding this comment

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

cool and direction-approved

@deltanedas deltanedas merged commit 5a32673 into DeltaV-Station:master Sep 17, 2024
13 checks passed
@beck-thompson beck-thompson deleted the limit-die-objective branch September 17, 2024 05:51
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants