Skip to content

update to minecraft 1.21.7, gradle to 8.14, TextPlaceholderAPI (use new methods)#341

Merged
John-Paul-R merged 3 commits into
1.21.xfrom
textparserapi-upgrade-off-deprecations
Jul 9, 2025
Merged

update to minecraft 1.21.7, gradle to 8.14, TextPlaceholderAPI (use new methods)#341
John-Paul-R merged 3 commits into
1.21.xfrom
textparserapi-upgrade-off-deprecations

Conversation

@John-Paul-R

Copy link
Copy Markdown
Owner

No description provided.

@semanticdiff-com

semanticdiff-com Bot commented Jul 9, 2025

Copy link
Copy Markdown

Review changes with  SemanticDiff

Changed Files
File Status
  src/main/java/com/fibermc/essentialcommands/commands/MotdCommand.java  41% smaller
  src/main/java/com/fibermc/essentialcommands/text/ECTextImpl.java  33% smaller
  gradle.properties Unsupported file format
  gradle/wrapper/gradle-wrapper.jar Unsupported file format
  gradle/wrapper/gradle-wrapper.properties Unsupported file format
  gradlew Unsupported file format
  gradlew.bat Unsupported file format

Comment on lines +125 to +127
var ret = Text.empty();
parsedText.getWithStyle(specifiedStyle).forEach(ret::append);
return ret;

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

potentially worth checking that there was no reason I was avoiding this (e.g. prestyled text that's supposed to cascade down to children still) and that there's no reason to avoid this for the "one node" case...

@John-Paul-R John-Paul-R merged commit 7bb35c6 into 1.21.x Jul 9, 2025
1 check passed
@John-Paul-R John-Paul-R deleted the textparserapi-upgrade-off-deprecations branch July 9, 2025 14:08
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