Skip to content

Plain text component language codes#1077

Merged
Thorinwasher merged 6 commits intoMockBukkit:v1.21from
Thorinwasher:plain-text-component-language-codes
Aug 30, 2024
Merged

Plain text component language codes#1077
Thorinwasher merged 6 commits intoMockBukkit:v1.21from
Thorinwasher:plain-text-component-language-codes

Conversation

@Thorinwasher
Copy link
Copy Markdown
Contributor

@Thorinwasher Thorinwasher commented Aug 16, 2024

Description

Fixes #1040

I might need to add more tests to this. But I honestly don't understand how translatable components work (got one more test from willkroboth)

Checklist

The following items should be checked before the pull request can be merged.

  • Code follows existing style.
  • Unit tests added (if applicable).

Comment thread src/main/java/be/seeseemelk/mockbukkit/adventure/Language.java
Comment thread src/main/java/be/seeseemelk/mockbukkit/adventure/Languages.java
@@ -0,0 +1,6886 @@
{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Where does this file come from? Is it available in paper?

Copy link
Copy Markdown
Contributor Author

@Thorinwasher Thorinwasher Aug 22, 2024

Choose a reason for hiding this comment

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

Generated by metaminer (legal grayzone), let me make a pr for that actually already done apparently

@Thorinwasher Thorinwasher force-pushed the plain-text-component-language-codes branch from 5502951 to 8b26f10 Compare August 30, 2024 09:51
@Thorinwasher Thorinwasher enabled auto-merge (squash) August 30, 2024 09:51
@sonarqubecloud
Copy link
Copy Markdown

@Thorinwasher Thorinwasher merged commit 1e31228 into MockBukkit:v1.21 Aug 30, 2024
willkroboth added a commit to CommandAPI/CommandAPI that referenced this pull request Sep 23, 2024
Updated test toolkit's MockBukkit version to include MockBukkit/MockBukkit#1077, which allows accessing Minecraft Language codes.

Moved utilities for creating mock argument parsers into ArgumentUtilities class.

Made example project build script run maven tests, so the tests in the `automated-tests` and `automated-tests-shaded` example projects are run by GitHub Actions.
willkroboth added a commit to CommandAPI/CommandAPI that referenced this pull request Oct 28, 2024
Updated test toolkit's MockBukkit version to include MockBukkit/MockBukkit#1077, which allows accessing Minecraft Language codes.

Moved utilities for creating mock argument parsers into ArgumentUtilities class.

Made example project build script run maven tests, so the tests in the `automated-tests` and `automated-tests-shaded` example projects are run by GitHub Actions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adventure's PlainTextComponentSerializer does not translate Minecraft Language Codes

3 participants