Skip to content
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

1.20.6/1.21.1 support, quiver functionality changed and huge chores #124

Merged
merged 5 commits into from
Sep 25, 2024

Conversation

FN-FAL113
Copy link
Owner

@FN-FAL113 FN-FAL113 commented Sep 25, 2024

Changes

  • 1.20.6/1.21.1 support, 1.21.1 is tested on sf chinese fork but will not work yet on official
  • minimum mc supported version from 1.16 to 1.17
  • prevent auto ladder from placing more ladders when relative block above/below is air
  • mystery sticks now has randomized entity potion effects instead of hardcoding them per stick.
  • added versioned util classes for enchantments, potion effect and class types that are missing from slimefun's versioned classes.
  • removed unused dependencies (findbugs) and use built-in annotations from java 17. Lombok setters/getters are manually implemented instead.
  • removed unused pdc classes for now
  • utilize bukkit's util consumer (deprecated) for now to prevent 1.21 java util consumer conflict.
  • moved consumers out of World#Spawn with same reason as former
  • quiver functionality has been changed to support minimum to max mc versions. Arrows will be drawn from the quiver on player bow shoot. Infinity bows will not auto draw arrows but requires manually adding an arrow in the inventory. Quivers will not work when player is over-encumbered (full inventory), there should be space for arrows being drawn.
  • huge chores (I don't wanna cause headaches to locale maintainers due to these changes but I gotta reduce the cringe with stuffs I wrote years ago)

Related Issues

  • TBD

Checklist

  • I have fully tested the proposed changes and promise that they will not break world generation, mob spawning, and the like.
  • I followed the existing code standards and didn't mess up the formatting.
  • I did my best to add documentation to any public classes or methods I added.
  • I have added Nonnull and Nullable annotations to my methods to indicate their behaviour for null values

- minimum mc supported version from 1.16 to 1.17
- removed unused dependencies (findbugs) and use built-in annotations from java 17. Lombok setters/getters are manually implemented instead.
- prevent auto ladder from placing more ladders when relative block above/below is air
- added versioned util classes for enchantments, potion effect and class types that are missing from slimefun's versioned classes.
- mystery sticks now has randomized entity potion effects instead of hardcoding them per stick.
- removed unused pdc classes for now
- utilize bukkit's util consumer (deprecated) for now to prevent 1.21 java util consumer conflict.
- moved consumers out of World#Spawn with same reason as former
- Quiver functionality has been changed to support minimum to max mc versions. Arrows will be drawn from the quiver on player bow shoot. Infinity bows will not auto draw arrows but requires manually adding an arrow to the inventory. Quivers will not work when player is over-encumbered (full inventory), there should be space for arrows being drawn.
- huge chores (I do no want to cause headaches for locale maintainers due to these changes but I gotta reduce the cringe with stuffs I wrote years ago)
@FN-FAL113 FN-FAL113 merged commit dcc11da into main Sep 25, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant