-
-
Notifications
You must be signed in to change notification settings - Fork 368
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
Add the variable case node instead of being hidden #5674
Conversation
bd134d0
to
3f08853
Compare
Little note on this but it may be out-of-scope yet it's very related, for flexibility this shouldn't convert variables to lowercase when set to true INSTEAD it should do equalsIgnoreCase otherwise when toggling it on/off this will cause lots of issues as it does permanent damage/change. Adding to that, this PR doesn't really fix the mentioned issue, this only shows the option but the behavior is still as described above. |
2b33c36
to
07fa2f1
Compare
Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
Description
The configuration node
case-insensitive variables: true
is hidden, so this pull requests makes it not hidden. See related issue for a valid reasoning.Target Minecraft Versions: any
Requirements: none
Related Issues: #2163