-
Notifications
You must be signed in to change notification settings - Fork 331
Conversation
Forgot to update this version number
Summary by CodeRabbit
- Chores
- Upgraded the application's Java language level across subprojects to the latest version. This modernizes the build environment and aligns with current platform standards.
- No user-facing changes or behavior differences are expected.
- Self-hosted users may need a compatible Java runtime to build or run the application.
WalkthroughUpdated the Java language version target in Gradle configuration from 24 to 25 for all subprojects. Changes
Estimated code review effort2 (Simple) | ~10 minutes Pre-merge checks and finishing touchesPassed checks (3 passed)
Finishing touchesGenerate unit tests
Tip Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks: custom_checks: - name: "Undocumented Breaking Changes" mode: "warning" instructions: | Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal). Please share your feedback with us on this Discord post. 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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Nitpick comments (1)
build.gradle (1)
61-65: Optional: make the toolchain version configurable via gradle.properties.Avoid hardcoding 25 so future bumps are a one-line change.
Apply this pattern:
In gradle.properties:
java.toolchain.version=25In build.gradle (replace the hardcoded value):
javaCompiler = javaToolchains.compilerFor {
languageVersion = JavaLanguageVersion.of(findProperty("java.toolchain.version") as String)
}
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Files selected for processing (1)
build.gradle(1 hunks)
Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
- GitHub Check: Build and test on windows-latest
- GitHub Check: Build caffeine with snapshot
- GitHub Check: Build and test on ubuntu-latest
- GitHub Check: Build spring-framework with snapshot
- GitHub Check: Build and test on macos-latest
Additional comments (1)
build.gradle (1)
61-65: Verify JDK 25 support across toolchain, Error Prone, and CI
- Gradle wrapper is at 9.1.0--confirm it bundles toolchain support for Java 25
- No
org.gradle.java.installations.auto-downloadset--ensure CI or local builds can auto-download JDK 25- Error Prone Core 2.42.0 predates JDK 25--validate compatibility or bump to a newer version
- Code imports
com.sun.tools.javac.*extensively--run full test suite on JDK 25 to catch API breaks- No CI workflows found--add or update
.github/workflows/*to useactions/setup-javawithjava-version: 25
Codecov Report All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1303 +/- ## ========================================= Coverage 88.49% 88.49% Complexity 2476 2476 ========================================= Files 93 93 Lines 8214 8214 Branches 1617 1617 ========================================= Hits 7269 7269 Misses 474 474 Partials 471 471 View full report in Codecov by Sentry. New features to boost your workflow:
|