Skip to content

Config Commands#2051

Merged
AzureAaron merged 14 commits intoSkyblockerMod:mainfrom
kevinthegreat1:config-commands
Apr 21, 2026
Merged

Config Commands#2051
AzureAaron merged 14 commits intoSkyblockerMod:mainfrom
kevinthegreat1:config-commands

Conversation

@kevinthegreat1
Copy link
Copy Markdown
Collaborator

Allows you to modify the config with commands.

@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Jan 19, 2026
@kevinthegreat1
Copy link
Copy Markdown
Collaborator Author

This was tested. Only booleans are supported for now. Should be pretty simple to review.

@Alex33856 Alex33856 added this to the 6.0.0 milestone Jan 22, 2026
Comment thread src/main/java/de/hysky/skyblocker/config/SkyblockerConfigManager.java Outdated
Comment thread src/main/java/de/hysky/skyblocker/config/SkyblockerConfigManager.java Outdated
@LifeIsAParadox LifeIsAParadox added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Jan 24, 2026
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed changes requested This PR need changes labels Jan 25, 2026
@AzureAaron AzureAaron added the new feature This issue or PR is a new feature label Jan 28, 2026
Copy link
Copy Markdown
Collaborator

@viciscat viciscat left a comment

Choose a reason for hiding this comment

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

Small nitpick i guess, not gonna request changes since it's minor and it isn't that important from avoiding duplicate code a bit

Comment thread src/main/java/de/hysky/skyblocker/config/ConfigCommands.java Outdated
@Alex33856 Alex33856 removed this from the 6.0.0 milestone Jan 30, 2026
@Alex33856 Alex33856 added merge conflicts This PR has merge conflicts that need solving. and removed reviews needed This PR needs reviews labels Feb 5, 2026
# Conflicts:
#	src/main/java/de/hysky/skyblocker/DisableAll.java
#	src/main/java/de/hysky/skyblocker/config/ConfigNullFieldsFix.java
#	src/main/java/de/hysky/skyblocker/config/SkyblockerConfigManager.java
#	src/main/java/de/hysky/skyblocker/skyblock/item/HotbarSlotLock.java
#	src/main/java/de/hysky/skyblocker/skyblock/item/ItemProtection.java
#	src/main/resources/assets/skyblocker/lang/en_us.json
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed merge conflicts This PR has merge conflicts that need solving. labels Apr 12, 2026
@kevinthegreat1
Copy link
Copy Markdown
Collaborator Author

Should be good now.

@AzureAaron AzureAaron added this to the 6.4.0 milestone Apr 20, 2026
@LifeIsAParadox LifeIsAParadox added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Apr 21, 2026
@Alex33856 Alex33856 requested a review from AzureAaron April 21, 2026 02:16
@AzureAaron AzureAaron merged commit 545a9c3 into SkyblockerMod:main Apr 21, 2026
2 checks passed
@LifeIsAParadox LifeIsAParadox removed the merge me please Pull requests that are ready to merge label Apr 21, 2026
Alex33856 pushed a commit that referenced this pull request Apr 21, 2026
* Add boolean config commands

* Clean up null fields fix

* Intellij is being weird with indents

* Clean up more config stuff

* Add feedback and refactor

* Add config query

* Fix spotless

* Update commands

* Solve merge conflicts

* Fix config patches

* Rename command again

* Separate command registration

* Change formatting of feedback messages

---------

Co-authored-by: alex <8379108+Alex33856@users.noreply.github.com>

(cherry picked from commit 545a9c3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature This issue or PR is a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants