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

Added bestiary lore from mainhand #51

Merged
merged 3 commits into from
Mar 8, 2024
Merged

Conversation

Wembler23
Copy link
Collaborator

run /bestiary lore frommainhand to copy the lore from your mainhand onto the mob's lore. 90% of this change is just refactoring everything to be components rather than strings.

run /bestiary lore frommainhand <mobname> to copy the lore from your mainhand onto the mob's lore. 90% of this change is just refactoring everything to be components rather than strings
backwards compatibility for already existing lore strings in the file
@@ -16,6 +16,7 @@
import java.util.ArrayList;
import java.util.List;
import net.kyori.adventure.text.Component;
import net.kyori.adventure.text.TextComponent;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know why the review dog isn't properly reporting these anymore - this unused import needs to be removed.

@Wembler23 Wembler23 added the ready Pull request is ready to merge label Mar 8, 2024
@Combustible Combustible merged commit e5008b0 into master Mar 8, 2024
4 checks passed
@Combustible Combustible deleted the bestiarylorecommandupdate branch March 8, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Pull request is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants