Merge pull request 'CI: setup-java v5 and Gradle build cache' (#20 ) from ci-caching into master

Reviewed-on: #20
Run the compile check on pull requests only, not on merges
2026-09-26 02:10:44 +00:00 · 2026-09-26 03:48:54 +02:00 · 2026-09-26 03:19:11 +02:00
3 changed files with 14 additions and 7 deletions
@@ -1,12 +1,11 @@
 name: Check

-# Compiles every Minecraft version on pull requests and on pushes to master,
-# so a change that breaks one version (e.g. a missed or bad port) fails before
-# it's merged instead of in the release build.
+# Compiles every Minecraft version on pull requests, so a change that breaks
+# one version (e.g. a missed or bad port) fails before it's merged instead of
+# in the release build. Merges to master aren't re-checked: the PR already
+# checked that code.
 on:
  pull_request:
-  push:
-    branches: [master]

 jobs:
  compile:
@@ -17,8 +16,11 @@ jobs:
        with:
          submodules: true

+      # The JDK comes from the runner's tool cache. `cache: gradle` stores Gradle's
+      # dependencies, Loom's Minecraft cache and the build cache in the runner's
+      # actions cache.
      - name: Set up Java 21
-        uses: actions/setup-java@v4
+        uses: actions/setup-java@v5
        with:
          distribution: temurin
          java-version: 21
@@ -44,8 +44,11 @@ jobs:
          fi
          echo "Building Saturn Client $MOD_VERSION"

+      # The JDK comes from the runner's tool cache. `cache: gradle` stores Gradle's
+      # dependencies, Loom's Minecraft cache and the build cache in the runner's
+      # actions cache.
      - name: Set up Java 21
-        uses: actions/setup-java@v4
+        uses: actions/setup-java@v5
        with:
          distribution: temurin
          java-version: 21
@@ -1,5 +1,7 @@
 org.gradle.jvmargs=-Xmx3G
 org.gradle.parallel=true
+# Reuse task outputs (e.g. compiled classes) when inputs haven't changed.
+org.gradle.caching=true

 # IntelliJ IDEA is not yet fully compatible with configuration cache, see: https://github.com/FabricMC/fabric-loom/issues/1349
 org.gradle.configuration-cache=false
Author	SHA1	Message	Date
selimaj-dev	04e7064790	Merge pull request 'CI: setup-java v5 and Gradle build cache' (#20 ) from ci-caching into master Reviewed-on: #20	2026-09-26 02:10:44 +00:00
selimaj-devandclaude	ff0bce4d24	Run the compile check on pull requests only, not on merges Check / compile (pull_request) Successful in 10m49s Details A merge to master re-ran the same compile the PR had just done. Keep the pull_request trigger and drop the push-to-master one. Co-Authored-By: Claude Opus 5.5 <[email protected]>	2026-09-26 03:48:54 +02:00
selimaj-devandclaude	2ecb4f80ea	Move setup-java to v5 and enable Gradle's build cache Check / compile (pull_request) Successful in 9m39s Details - setup-java v4 is deprecated; use v5 (still with `cache: gradle`, which works once job containers can reach the runner's cache server). - Enable Gradle's build cache (org.gradle.caching) so unchanged compilations are restored from cache instead of re-run. Co-Authored-By: Claude Opus 5.5 <[email protected]>	2026-09-26 03:19:11 +02:00