Skip to content

Fix release Java selection and cached CodeQL builds - #495

Merged
SkyBlade1978 merged 2 commits into
MinecraftModDevelopmentMods:master-1.12from
SkyBlade1978:master-1.12
Oct 1, 2026
Merged

SkyBlade1978 merged 2 commits into
MinecraftModDevelopmentMods:master-1.12from
SkyBlade1978:master-1.12

Conversation

@SkyBlade1978

@SkyBlade1978 SkyBlade1978 commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Two workflow problems surfaced during final release checks.

  • The 1.12 build requires Temurin 17.0.1+12, but the release action selected the latest Java 17 when switching away from the Java 8 game toolchain. Use the exact selector for both release builds and Maven publication. The 1.18 route keeps its existing target toolchain.
  • CodeQL failed when Gradle restored compiled classes from its build cache. Disable build-cache reuse in the scan's compile step so CodeQL observes a real javac run.

The new workflow checks failed against the old settings and pass with these fixes. A clean no-cache compilation passes locally, along with check, build, Javadocs, release artifact auditing, and Eclipse isolation. This PR changes only workflows and their checks; gameplay, dependency versions, and release version are unchanged.

Use Temurin 17.0.1+12 for both release builds and Maven publication, matching the existing build guard. Add a workflow check so the release selector cannot drift back to a floating Java 17 version. The new check failed before the change and passes afterward.
Disable Gradle build-cache reuse in the CodeQL compile step so the scanner observes javac, even when production sources have not changed. The workflow guard reproduces the missing setting and passes with the fix; a clean no-cache compilation passes too.
@SkyBlade1978 SkyBlade1978 changed the title Pin Java 17 for the Base Metals 1.12 release action Fix release Java selection and cached CodeQL builds Oct 1, 2026
@SkyBlade1978
SkyBlade1978 merged commit 9edbb6f into MinecraftModDevelopmentMods:master-1.12 Oct 1, 2026
5 checks passed
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