Skip to content

Convert to config#22

Merged
Ruling-0 merged 19 commits intomasterfrom
Convert-to-config
Feb 28, 2026
Merged

Convert to config#22
Ruling-0 merged 19 commits intomasterfrom
Convert-to-config

Conversation

@LazyFleshWasTaken
Copy link
Copy Markdown

Converts wand capacity and caps to config so changes are not hardcoded.

Tested in devenv, changes to config are properly changed in game.

image image

@LazyFleshWasTaken
Copy link
Copy Markdown
Author

Draft because I don't know how to make the thaumonomicon pages, which use the lang file entry, show the config value instead of the hardcoded.
If that isn't a problem, can be undrafted ig

@koolkrafter5 koolkrafter5 self-assigned this Feb 8, 2026
@koolkrafter5 koolkrafter5 added the enhancement New feature or request label Feb 12, 2026
@koolkrafter5 koolkrafter5 marked this pull request as ready for review February 12, 2026 08:54
@koolkrafter5 koolkrafter5 requested a review from a team February 12, 2026 08:54
@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Feb 12, 2026
@koolkrafter5 koolkrafter5 removed their assignment Feb 13, 2026
Copy link
Copy Markdown

@Ruling-0 Ruling-0 left a comment

Choose a reason for hiding this comment

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

Looks good overall, just some minor points.

Comment thread src/main/java/fox/spiteful/forbidden/compat/BloodMagic.java Outdated
Comment thread src/main/java/fox/spiteful/forbidden/items/ForbiddenItems.java Outdated
Comment thread src/main/java/fox/spiteful/forbidden/Config.java
Comment thread src/main/java/fox/spiteful/forbidden/Config.java
Comment thread src/main/resources/assets/forbidden/lang/en_US.lang Outdated
@Ruling-0 Ruling-0 merged commit b65ce08 into master Feb 28, 2026
1 check passed
@Ruling-0 Ruling-0 deleted the Convert-to-config branch February 28, 2026 04:46
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants