Skip to content

chore/fix: compare patch version in MinecraftVersion#4877

Open
PierreSchwang wants to merge 1 commit into
mainfrom
chore/minecraftVersions
Open

chore/fix: compare patch version in MinecraftVersion#4877
PierreSchwang wants to merge 1 commit into
mainfrom
chore/minecraftVersions

Conversation

@PierreSchwang
Copy link
Copy Markdown
Member

@PierreSchwang PierreSchwang commented May 25, 2026

Overview

Follow-up to #4871

Description

I've added a field for the patch version, but did not compare it yet (added afterwards, as I did not know that the newer versions actually have patch versions). Given patch versions are not relevant (yet), the current code is not inherently broken - just not fully correct.

  • Added documentation on how (and if) patch version parts are provided.
  • Use compareTo for object comparisions.
  • Validate version parts (non-negative).
  • Added basic tests for comparison methods (compareTo is implicitly tested).

Submitter Checklist

  • Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
  • Ensure that the pull request title represents the desired changelog entry.
  • New public fields and methods are annotated with @since TODO.
  • I read and followed the contribution guidelines.

@PierreSchwang PierreSchwang requested a review from a team as a code owner May 25, 2026 20:13
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