Modernize Gradle Build System #150
Conversation
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
- Introduced Gradle Version Catalog in `gradle/libs.versions.toml`. - Centralized repository configuration in `settings.gradle.kts` using `dependencyResolutionManagement`. - Updated all subprojects to use the Version Catalog. - Converted `cake/core/build.gradle` to Kotlin DSL. - Cleaned up `gradle.properties` by removing external dependency versions. - Restored `nativeimage.gradle` inclusion in `cake/lwjgl3/build.gradle`. - Removed redundant `kotlin-stdlib` dependency from `cake/core`. - Verified the build and tests pass successfully.
demoth
force-pushed
the
modernize-gradle-setup-v2-13061407244758310799
branch
from
3d47096 to
6ff082d
Compare
- Updated Gradle wrapper to 8.14. - Upgraded Java source and target compatibility to 21. - Updated Kotlin JVM target to 21. - Simplified `cake/lwjgl3/build.gradle` by removing redundant Java 11/14 compatibility checks. - Fixed `duplicatesStrategy` deprecation warning in `cake/lwjgl3`. - Verified build and tests pass with Java 21. Co-authored-by: demoth <638804+demoth@users.noreply.github.com>
…rnings - Updated Gradle wrapper to 8.12.1. - Set Java baseline to 21 across all modules. - Upgraded Kotlin to 2.1.0 and Coroutines to 1.9.0. - Centralized dependency management via Version Catalog and settings.gradle.kts. - Fixed Kotlin compiler warning in `func_object.kt` by updating `registerTouch` signature. - Suppressed expected deprecation warnings in tests and legacy client code. - Verified build and tests pass. Co-authored-by: demoth <638804+demoth@users.noreply.github.com>
- Replaced the custom `Vargs` class with native Java varargs (`Object...`) in `jake2.qcommon.Com`. - Updated all callers of `Printf`, `DPrintf`, `Error`, and `sprintf` to use the new native varargs syntax. - Removed the deprecated `Vargs` class and all its imports. - Cleaned up legacy commented-out code that used `Vargs`. - Verified the build and tests pass successfully. Co-authored-by: demoth <638804+demoth@users.noreply.github.com>
1 participant
This change modernizes the Gradle build system of the Jake2 project. It introduces a Version Catalog to centralize dependency management, moves repository configuration to
settings.gradle.ktsfor better structure, and updates various build scripts to use these new features. It also includes some cleanup of redundant configurations and ensures that specialized builds like GraalVM native image support remain intact.PR created automatically by Jules for task 13061407244758310799 started by @demoth