melix
02/15/2022, 3:07 PM.all
, not .configureEach
. Let me explain why. https://github.com/gradle/gradle/blob/master/subprojects/jacoco/src/main/java/org/gradle/testing/jacoco/plugins/JacocoReportAggregationPlugin.java#L98melix
02/15/2022, 3:08 PMreports._register_
, which is lazy.melix
02/15/2022, 3:08 PMmelix
02/15/2022, 3:08 PMVampire
02/15/2022, 3:09 PMmelix
02/15/2022, 3:09 PMcreate
instead of register (it probably should), but from the plugin point of view, I think semantically, what you want is really: for each report in the model, register a task. So the model must be eager, but the tasks must be lazy.lptr
02/15/2022, 3:49 PMkyle
02/15/2022, 4:21 PMkyle
02/15/2022, 4:22 PMmelix
02/15/2022, 4:58 PMRodrigo Oliveira
02/18/2022, 1:18 PMregister
and there should be a consumer at a definite point in time.
Making everything eager now (core and plugins) goes against the configuration avoidance discipline we’ve been working to establish.Vampire
02/18/2022, 1:25 PMkyle
02/18/2022, 3:50 PMkyle
02/18/2022, 3:50 PMmelix
02/18/2022, 3:52 PMdaniel
02/18/2022, 7:01 PMkyle
02/18/2022, 7:58 PM