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

Ensure Gradle tasks are configured lazily #1292

Closed
wants to merge 1 commit into from

Conversation

vpavic
Copy link
Contributor

@vpavic vpavic commented Dec 30, 2021

At present, generated Gradle build scripts that use Groovy DSL configure tasks eagerly. This is both suboptimal and actually not aligned with the generated build scripts that use Kotlin DSL, which configures tasks lazily.

This commit updates GroovyDslGradleBuildWriter to ensure tasks are configured lazily.

See https://docs.gradle.org/current/userguide/task_configuration_avoidance.html for more details on the task configuration avoidance topic.

At present, generated Gradle build scripts that use Groovy DSL configure tasks eagerly. This is both suboptimal and actually not aligned with the generated build scripts that use Kotlin DSL, which configures tasks lazily.

This commit updates `GroovyDslGradleBuildWriter` to ensure tasks are configured lazily.

See https://docs.gradle.org/current/userguide/task_configuration_avoidance.html for more details on the task configuration avoidance topic.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 4, 2022
@wilkinsona wilkinsona added this to the 0.12.0 milestone Jan 9, 2022
@wilkinsona wilkinsona added type: enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 9, 2022
@snicoll snicoll modified the milestones: 0.12.0, General Backlog Jan 21, 2022
@snicoll snicoll self-assigned this Jan 22, 2022
snicoll pushed a commit that referenced this pull request Jan 22, 2022
At present, generated Gradle build scripts that use Groovy DSL configure
tasks eagerly. This is both suboptimal and actually not aligned with the
generated build scripts that use Kotlin DSL, which configures tasks
lazily.

This commit updates `GroovyDslGradleBuildWriter` to ensure tasks are
configured lazily.

See gh-1292
snicoll added a commit that referenced this pull request Jan 22, 2022
@snicoll snicoll closed this in 97e5561 Jan 22, 2022
@vpavic vpavic deleted the gradle-task-config-avoidance branch January 24, 2022 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants