-
Notifications
You must be signed in to change notification settings - Fork 32
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
fix: obliterated disadvantage #519
base: main
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughIn this update, the implementation has been bifurcated across two key files. Within the object controller at Possibly related PRs
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🔇 Additional comments (5)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
@KRdaMystic i have no qualms with this but this is your area in general. If you have no issue with these changes i'll merge |
Purpose of the PR
Sieged
disadvantage provides high stock of spare equipment and gene-seed on game start.Obliterated
disadvantage however, does not. This PR focuses on addressing this.Describe the solution
Copy, paste and modify lines of code from
sieged
toobliterated
.Also tweak the current
sieged
disadvantage to also include techmarine and chaplain equipment.Implementing this ensures that
obliterated
disadvantage properly becomes a more extreme version ofsieged
.Describe alternatives you've considered
Create more advantages/disadvantages covering more situations and circumstances of player's starting situation in the game, such as:
Add a
depleted armamentarium
disadvantage which lowers or removes the additional starting equipment associated withsieged
and now possiblyobliterated
.Add a
depleted gene-seed stocks
disadvantage which lowers or removes the additional starting gene-seed associated withsieged
andobliterated
.Testing done
Related links
https://discord.com/channels/714022226810372107/1272865272159535114/1344229545875738697
Summary by CodeRabbit