Skip to content

Throw exception for CraftMagicNumbers#getMappingsVersion#12676

Merged
kennytv merged 1 commit intoPaperMC:dev/1.21.6from
JRoy:remove-md5-magic-string
Jun 17, 2025
Merged

Throw exception for CraftMagicNumbers#getMappingsVersion#12676
kennytv merged 1 commit intoPaperMC:dev/1.21.6from
JRoy:remove-md5-magic-string

Conversation

@JRoy
Copy link
Member

@JRoy JRoy commented Jun 15, 2025

after hard fork it makes no sense to have this. devs can use ServerBuildInfo#minecraftVersionId if they need a string that will change every version

@JRoy JRoy requested a review from a team as a code owner June 15, 2025 21:30
@github-project-automation github-project-automation bot moved this to Awaiting review in Paper PR Queue Jun 15, 2025
@github-project-automation github-project-automation bot moved this from Awaiting review to Awaiting final testing in Paper PR Queue Jun 15, 2025
Copy link
Contributor

@AlexProgrammerDE AlexProgrammerDE left a comment

Choose a reason for hiding this comment

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

Would be great to have this in for 1.21.6, so that plugin devs can properly fallback.

@kennytv kennytv merged commit 9dcbf0a into PaperMC:dev/1.21.6 Jun 17, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting final testing to Merged in Paper PR Queue Jun 17, 2025
@JRoy JRoy deleted the remove-md5-magic-string branch June 17, 2025 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

5 participants