Skip to content

Modernize Plugin Application and Build Directory#831

Open
gallardo wants to merge 2 commits intoalkacon:masterfrom
gallardo:chore/modernize-plugins-and-layout-PR1
Open

Modernize Plugin Application and Build Directory#831
gallardo wants to merge 2 commits intoalkacon:masterfrom
gallardo:chore/modernize-plugins-and-layout-PR1

Conversation

@gallardo
Copy link
Contributor

@gallardo gallardo commented Jan 8, 2026

Refactors build.gradle to use the modern plugins {} block and layout.buildDirectory API, replacing legacy and deprecated patterns. This PR has been split into two distinct commits to maintain a clean history.

Changes

  1. Modernize Plugin Application: Migrated apply plugin and buildscript configurations to the modern plugins {} block. This enables better classloader isolation and is a prerequisite for the Gradle Configuration Cache.
  2. Modernize Build Directory: Completely replaced the deprecated buildDir property with the modern layout.buildDirectory API. Adopted a cleaner syntax using layout.buildDirectory.dir("...") and layout.buildDirectory.file("...").

Justification:

  • Standards: Removes deprecated property usage and adopts modern Gradle idioms.
  • Maintainability: Cleaner, more declarative syntax for plugins and build paths.
  • Future-proofing: Essential foundation for upcoming performance improvements like the Configuration Cache.

Verification:

  • Verified that all main artifacts (JARs, dependencies) are correctly generated in the ../BuildCms directory.
  • Confirmed build configuration validity via ./gradlew tasks.
  • Confirmed tlddoc tasks are still correctly registered.

@gallardo gallardo marked this pull request as draft January 8, 2026 14:45
@gallardo gallardo force-pushed the chore/modernize-plugins-and-layout-PR1 branch from 2501139 to a26623b Compare January 8, 2026 16:06
@gallardo gallardo marked this pull request as ready for review January 8, 2026 16:20
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

Comments