stevesaliman / gradle-properties-plugin

Gradle plugin to simplify loading project properties from external environment specific files
Apache License 2.0
192 stars 28 forks source link

Adopt task configuration avoidance #42

Open Vampire opened 1 year ago

Vampire commented 1 year ago

Please adopt task configuration avoidance. It is present for a long time already and your plugin completely voids its time savings. For example it uses tasks.all instead of tasks.configureEach which causes each and every task in the build to be realized and configured. See https://docs.gradle.org/current/userguide/task_configuration_avoidance.html for more information.

.all might indeed by the only violation, at least from a very quick check I didn't see other violations. You can also use Gradle build scans to see which tasks were realized at configuration time wich should optimally be 0.