Skip to content

Conversation

@timoreo22
Copy link
Member

Avoids the "version is 10.0.0 and newest is 10.0.0" bug, it will come back if we switch back to two version numbers.

let latest_version_parts = semver_split(&latest_version);

if log_version_parts.len() != 2
if log_version_parts.len() != 3
Copy link
Member

Choose a reason for hiding this comment

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

What would happen if the len is 2 as in 9.4?

Copy link
Member Author

Choose a reason for hiding this comment

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

If len is 2, then the branch is taken and the outdated message is shown

it's basically what's currently happening with 3, depite having the same numbers.

Copy link
Member

Choose a reason for hiding this comment

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

Then it'd say that it's outdated. which makes sense

@TheKodeToad TheKodeToad merged commit deab165 into PrismLauncher:main Jan 7, 2026
10 checks 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.

4 participants