Skip to content

Bump slf4j-api to match Terasology - #88

Open
soloturn wants to merge 1 commit into
preview/v4.x-SNAPSHOTfrom
bump-slf4j-logback
Open

Bump slf4j-api to match Terasology#88
soloturn wants to merge 1 commit into
preview/v4.x-SNAPSHOTfrom
bump-slf4j-logback

Conversation

@soloturn

Copy link
Copy Markdown

AI-assisted change proposal.

Bumps slf4j-api 1.7.30 -> 2.0.18 in nui, nui-reflect, and nui-gestalt, matching Terasology's own pin and gestalt's recent bump (MovingBlocks/gestalt#176).

No logback dependency exists in this repo, and no Android module is on this branch (dropped per maintainer direction in #83), so unlike gestalt there's no SLF4J-2.x-vs-old-backend compatibility fallout to work around here - a plain version bump.

Verified: ./gradlew build passes, and slf4j-api resolves cleanly to 2.0.18 everywhere (dependencyInsight shows the transitive 1.7.25 request from gestalt-di/gestalt-inject losing conflict resolution to it, no strict-constraint conflicts).

1.7.30 -> 2.0.18 in nui, nui-reflect, nui-gestalt, matching gestalt's
recent bump (MovingBlocks/gestalt#176) and Terasology's own pin.
No Android module on this branch and no logback dependency, so no
compatibility fallout to work around.
@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 93a1aa9d-ebad-442b-ac4c-be866331c189


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.

@soloturn
soloturn changed the base branch from master to preview/v4.x-SNAPSHOT August 27, 2026 19:18
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.

1 participant