Skip to content

De-randomises Inquisition Enhancements - #7442

Open
budgetcommander0 wants to merge 14 commits into
Monkestation:mainfrom
budgetcommander0:enhancements
Open

De-randomises Inquisition Enhancements#7442
budgetcommander0 wants to merge 14 commits into
Monkestation:mainfrom
budgetcommander0:enhancements

Conversation

@budgetcommander0

Copy link
Copy Markdown

About The Pull Request

Removes randomisation from Inquisition enhancements, allowing players to pick from the full selection.

Why It's Good For The Game

While the idea of forcing players to pick from a random pool of enhancements is commendable, this is both unintuitive for new players ("Why is x option missing?") and disallows players to have a character with a "canon" enhancement. Ultimately, this system is confusing, hostile to the development of clear characters to roleplay, and - in my personal opinion - feels unnecessarily gamey in a faction that already feels unnecessarily gamey enough as-is.

Changelog

🆑
del: Removes enhancement randomisation for applicable Inquisition classes
/:cl:

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

De-randomises enhancements for the Argent
De-randomises enhancements for the Confessor
De-randomises Inspector enhancements
Fixes the parenthesis placement
@github-actions github-actions Bot added the Removal What used to be there is no longer there. How sad. label Jul 29, 2026
@ModDownloading

ModDownloading commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

@realforest2001 I saw you commented on the balance of the PR and I would like to say that even if folks don't have a meta option available they can still make a tier list of options and ignore the worst ones. I would rather balance out the options so they're all (or at least mostly) viable rather than giving people random chances to get stuff they actually like. I'm willing to make changes myself if any one option is picked way too much!

@ModDownloading

Copy link
Copy Markdown
Collaborator

@budgetcommander0 Could you add the Disguise Kit option to be available for the Sacrestants again? You seem to have removed it from them only.

@budgetcommander0

Copy link
Copy Markdown
Author

I'll get on that

@budgetcommander0

Copy link
Copy Markdown
Author

@budgetcommander0 Could you add the Disguise Kit option to be available for the Sacrestants again? You seem to have removed it from them only.

Done.

@realforest2001

Copy link
Copy Markdown
Collaborator

As with the original PR, needs balance approval due to implications of being able to pick meta options every time.

@realforest2001 realforest2001 added the Balance Surely this is good for the game label Jul 31, 2026

@ModDownloading ModDownloading left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks all good to me! I'm not a balance maintainer though so it will still need approval from them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Balance Surely this is good for the game Removal What used to be there is no longer there. How sad.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants