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.

Sign up for GitHub

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

Jump to bottom

simplify #469

Draft
wants to merge 22 commits into
base: master
Choose a base branch
from
Draft

simplify #469

wants to merge 22 commits into from

Conversation

CJCrafter
Copy link
Member

@CJCrafter CJCrafter commented Oct 12, 2024 •
edited
Loading

removes all version compatibility code. Only target latest stable API, avoid all else.

  • Remove old compatibility code before 1.21
  • Remove all old lib .jar files. Depend only on maven repos! Finally!
  • Update serializer stuff
    • Add support for Bukkit registers
    • Improve list parsing
    • Convert MechanicsCore serializers
    • Convert WeaponMechanics serializers/validators
  • Remove command compatibility
    • Add Jorel's command api
    • Convert commands to jorel
    • add custom arguments for map serialization
  • Update skin stuff
    • Remove support for ALL old systems. Custom model data or bust.
    • Figure out this new model system
    • Update the resource pack
  • Update durability stuff
    • Ditch old system completely, automatically convert item so it has a durability bar
    • Let the item fall to 0 durability (configurable)
  • Switch to PacketEvents

@CJCrafter
remove old compatibility versions
819777b
@CJCrafter
remove 1.12 compatible commands
b0071be
@CJCrafter
remove "devMode" since no old versions are used anymore
9ef2eed
@CJCrafter
Merge branch 'refs/heads/master' into cj/simplify
69d4d7f 
# Conflicts:
#	BuildMechanicsCore/build.gradle.kts
#	BuildWeaponMechanics/build.gradle.kts
@CJCrafter
migrate version checking and reflection to new ReflectionUtil
bb2be0b
@CJCrafter
start conversion to jorel's command api
9623800
@CJCrafter
simplify test cases, because MockBukkit is above my pay grade
ab5d33c
@CJCrafter
Remove all old lib files
482952b
@CJCrafter
fix list format getting an extra '<>'
3733485
@CJCrafter
update serializer exceptions to be a bit more concise
217a9bb
@CJCrafter
update serializers to new optional system
d6f3420
@CJCrafter
update to gradle 8.11.1 and paperweight 2.0.0
6b9c96a
@CJCrafter
update weaponmechanics serializers to new optional system
8d1741f
@CJCrafter
spotlessApply
c00c70a
@CJCrafter
convert weaponmechanics commands to jorel's command api
eb65b09
@CJCrafter
convert the rest of the weapon mechanics serializers
d3f0968
@CJCrafter
update attributes to the new system. Add support for operations
c16dd72
@CJCrafter
remove support for "Broken_Item"
d4ad8f1
@CJCrafter
remove unused compatibility methods
22ba032
@CJCrafter
switch to packetevents
a7163ac
@CJCrafter
fix bstats not being included
fe069a2
@CJCrafter
first build. Update libs, add compatibility for 1.21.4
a533d4c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DeeCaaD DeeCaaD Awaiting requested review from DeeCaaD DeeCaaD will be requested when the pull request is marked ready for review DeeCaaD is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@CJCrafter