Skip to content

Conversation

WaitingIdly
Copy link
Collaborator

@WaitingIdly WaitingIdly commented Oct 8, 2025

changes in this PR:

  • update GregTechCEu/Buildscripts.
    • ensure that spotless stays using 6.23.0 and not the buildscript default of 6.13.0 (jabel breaks 6.13.0).
  • move all build.gradle exclusive properties to the dedicated buildscript.properties file.
  • remove unneeded project. references from dependencies.gradle.
  • sort the mod_dependencies map alphabetically by key instead of sorting by first value in value list (insane).
    • also adjust mod_dependencies to be more like runtime_dependencies.
  • update mixinbooter from 9.1 to 10.7, fixing javadocs for mixin features and adding additional mixin options.
  • rename some of the gradle.properties settings - mods have their full name and the non-mod "debug" settings are "dev".
  • add -Dterminal.jline=true to additionalJavaArguments, making the terminal more colorful (warnings/errors easier to spot)

@WaitingIdly WaitingIdly added the build-script Something in build scripts changed label Oct 8, 2025
Copy link
Member

@brachy84 brachy84 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:clueless:

@MCTian-mi
Copy link

what about enabling java17/21 runTasks, since you updated spotless

@brachy84 brachy84 merged commit a19b7a5 into CleanroomMC:master Oct 9, 2025
@WaitingIdly WaitingIdly deleted the update-buildscript branch October 9, 2025 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-script Something in build scripts changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants