Skip to content

fix: use Enchantment#description() for MiniMessage enchantment tag#697

Open
ybw0014 wants to merge 1 commit intopylonmc:masterfrom
ybw0014:fix/rebar-mini-message
Open

fix: use Enchantment#description() for MiniMessage enchantment tag#697
ybw0014 wants to merge 1 commit intopylonmc:masterfrom
ybw0014:fix/rebar-mini-message

Conversation

@ybw0014
Copy link
Copy Markdown
Contributor

@ybw0014 ybw0014 commented Apr 11, 2026

Not all enchantments have a translation key. Using Enchantment#translationKey() can cause error if the enchantment description is not translatable (like in RebarMobs, all the Enchantment descriptions are simple text components)

@ybw0014 ybw0014 changed the title fix: use Enchantment#description() fix: use Enchantment#description() for MiniMessage enchantment tag Apr 11, 2026
@ybw0014
Copy link
Copy Markdown
Contributor Author

ybw0014 commented Apr 11, 2026

is CI failing matter for this PR?

@LordIdra
Copy link
Copy Markdown
Contributor

no, just a flaky test

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.

3 participants