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

make thief a subgamemode #25740

Merged
merged 8 commits into from
Mar 2, 2024

Conversation

deltanedas
Copy link
Contributor

@deltanedas deltanedas commented Mar 1, 2024

About the PR

thing ripped out of turf war so it can be merged now

Why / Balance

adding rule components for 1 gamemode onto another gamemode is evil

Technical details

see other pr

Media

the smite still works
01:56:00

adding game rule is real
02:01:09

roundstart random thief rule works, on traitor preset
02:14:16

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

Breaking changes

ruleChance removed from ThiefRule, which is no longer added to random gamemodes' rules. use the SubGamemodes component instead.

Changelog
no cl no fun

@github-actions github-actions bot added the S: Needs Review Status: Requires additional reviews before being fully accepted label Mar 1, 2024
@metalgearsloth
Copy link
Contributor

image
what de fok

@metalgearsloth
Copy link
Contributor

At some point this just needs to be dynamic but better than whatever got merged before.

@Brandon-Huu
Copy link
Contributor

Does this stop thieves from spawning on nukie rounds?

@deltanedas
Copy link
Contributor Author

deltanedas commented Mar 1, 2024

no its just refactor

also dont merge i havent verified it works im trolling

@deltanedas
Copy link
Contributor Author

nvm i just didnt opt in to thief :trollface::trollface::trollface::trollface::trollface:

@EmoGarbage404 EmoGarbage404 added the S: Awaiting Changes Status: Changes are required before another review can happen label Mar 2, 2024
@deltanedas deltanedas requested a review from EmoGarbage404 March 2, 2024 15:14
@github-actions github-actions bot removed the S: Awaiting Changes Status: Changes are required before another review can happen label Mar 2, 2024
@@ -93,6 +93,7 @@
showInVote: false
rules:
- Traitor
- SubGamemodes
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- SubGamemodes
- SubGamemodesRule

troll

Copy link
Contributor Author

Choose a reason for hiding this comment

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

phone webedit ops to the rescue

@EmoGarbage404 EmoGarbage404 merged commit 7f060eb into space-wizards:master Mar 2, 2024
10 checks passed
@deltanedas deltanedas deleted the subgamemode-ops branch March 2, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Needs Review Status: Requires additional reviews before being fully accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants