Skip to content

Conversation

@PuppyTransGirl
Copy link
Owner

@PuppyTransGirl PuppyTransGirl commented Jan 18, 2026

Close #84

@PuppyTransGirl PuppyTransGirl self-assigned this Jan 18, 2026
@PuppyTransGirl PuppyTransGirl added the feature New feature request or implementation label Jan 18, 2026
@PuppyTransGirl PuppyTransGirl added this to the 3.1.0 milestone Jan 18, 2026
@PuppyTransGirl PuppyTransGirl linked an issue Jan 18, 2026 that may be closed by this pull request
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

Warning

Rate limit exceeded

@PuppyTransGirl has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 17 minutes and 37 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 377d430 and a8cbfab.

📒 Files selected for processing (3)
  • src/main/java/toutouchien/niveriaapi/utils/BackwardUtils.java
  • src/main/java/toutouchien/niveriaapi/utils/ItemBuilder.java
  • src/main/java/toutouchien/niveriaapi/utils/VersionUtils.java
📝 Walkthrough

Walkthrough

This pull request adds version-awareness and backward-compatibility utilities and updates build configuration. It introduces a new VersionUtils enum for server version detection, a BackwardUtils class exposing UI_SOUND_SOURCE, and refactors ItemBuilder with version-guarded branches. Several interactive menu components switch their default click sound to BackwardUtils.UI_SOUND_SOURCE. Build changes add a minMinecraftVersion Gradle property, wire it into resource processing, and update paper-plugin.yml to use ${minMinecraftVersion}. Minor README formatting adjustments are also included.

🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 35.29% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed Title clearly describes the main change: adding support for Minecraft versions 1.21.4 through 1.21.11.
Description check ✅ Passed Description references the linked issue #84, relating to adding support for older Minecraft versions, which aligns with the changeset.
Linked Issues check ✅ Passed The PR successfully implements version detection utility and backward compatibility mechanisms to support multiple Minecraft versions, directly addressing the requirement to add support for older versions [#84].
Out of Scope Changes check ✅ Passed All changes are in-scope: version utility infrastructure, backward compatibility layer, multi-version resource filtering, and minor documentation updates directly support older version compatibility.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 4

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 3

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
build.gradle.kts (1)

86-91: Verify the Modrinth snapshot version is stable and published.
Snapshots can disappear and break local dev runs; consider pinning to a released version if possible.

@PuppyTransGirl PuppyTransGirl merged commit 70262ff into master Jan 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature request or implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for older Minecraft versions

2 participants