Skip to content

tweak how and when the 'v' prefix is applied by the update checker#965

Merged
gniftygnome merged 2 commits intoTerraformersMC:26.1from
JustPyrrha:26.1
Jan 21, 2026
Merged

tweak how and when the 'v' prefix is applied by the update checker#965
gniftygnome merged 2 commits intoTerraformersMC:26.1from
JustPyrrha:26.1

Conversation

@JustPyrrha
Copy link
Contributor

replaces #948

from original pr:
Some mods start their version strings with something other than a number or a 'v' prefix such as Sodium and Simple Voice Chat (examples below) which can make the 'v' prefix applied by the update checker look a bit messy.

This simply checks if the first character of a version string isn't a number (using !Character.isDigit) and appends .nonNumeric to the translation key which is a duplicate of the previous key without the 'v' prefix. Versions starting with a number aren't changed.

Sodium before:
Screenshot_20251224_165913

Sodium after:
Screenshot_20251224_165821

Simple Voice Chat before:
Screenshot_20251224_165919

Simple Voice Chat after:
Screenshot_20251224_165831

Fabric Api before and after:
Screenshot_20251224_170728

Copy link
Contributor

@gniftygnome gniftygnome left a comment

Choose a reason for hiding this comment

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

LGTM. This may cause some pain when merging #960, but I'll deal with it.

@gniftygnome gniftygnome merged commit 103ddb0 into TerraformersMC:26.1 Jan 21, 2026
1 check passed
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.

2 participants