Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Buildscripts with new SpongeGradle #2046

Closed
wants to merge 4 commits into from

Conversation

gabizou
Copy link
Member

@gabizou gabizou commented Sep 9, 2019

SpongeGradle | SpongeAPI | SpongeCommon | SpongeVanilla | SpongeForge

This is a working PR with the development lock step with SpongeGradle. While this technically is compatible with Gradle 4.10.3, I've elected to use the latest Gradle 5.6.2 (as of current writing) to take advantage of the improved Kotlin-DSL since those versions, and configuration avoidance throughout the build process.

Some changes:

  • Constants are just constants in files
  • Things that should've been made properties are now properties and/or functions in SpongeGradle or in the gradle.properties of their respective projects
  • Updated to use appropriate Gradle ideologies with configuration avoidance (lazy evaluation)
  • Reduced the overall build file size and using as much handling with the gradle plugins to configure most things we've otherwise duplicated and/or scripted with other scripts.

Signed-off-by: Gabriel Harris-Rouquette <gabizou@me.com>
Signed-off-by: Gabriel Harris-Rouquette <gabizou@me.com>
Signed-off-by: Gabriel Harris-Rouquette <gabizou@me.com>
Signed-off-by: Gabriel Harris-Rouquette <gabizou@me.com>
@gabizou gabizou closed this Sep 9, 2019
@gabizou gabizou deleted the build/kotlin-dsl branch October 8, 2019 18:17
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