Conversation
There was a problem hiding this comment.
Enchantments described in output aren't correct (though are applied correctly) - in my opinion should be a blocker to merging
Tested on 2.21.0-dev+99-07f97cf
Tested on Purpur build 2246 as well as Paper:
This server is running Paper version 1.21-37-master@dd49fba (2024-06-24T02:08:49Z) (Implementing API version 1.21-R0.1-SNAPSHOT)
You are running the latest version
Ignore riptide that was just testing
|
|
Should be fixed as of latest commit, was a CraftBukkit bug/change that causes Enchantment#getName to return the wrong name for enchantments. So for example, currently Enchantment.SHARPNESS will return RIPTIDE. Switched to using namespaced keys for 1.13+. |
Enchantment#getName is deprecated and gives the wrong names for enchantments starting in 1.21
|
Seems good to me - looks like the update check for feature branches doesn't work too well (didn't seem to recognise the new commit) but that is probably unrelated/out of scope for the PR |
I know this, when I enchanting breach, it shows Projectile Protect... |







really only waiting on item aliases and maybe easter eggs