Ksawery Karwacki
09/04/2024, 3:31 PMVampire
09/04/2024, 4:53 PMafterEvaluate with all its drawbacks. If the task of that plugin uses `Property`s but the extension primitives, I'd say that plugin should get its ass up and properly finish the migration to the lazy API. How to best mitigate the situation is very hard to say from the generic description you stated. But if the tasks indeed have `Property`s and you know it is enough to set those, it might well be the best way to go, yes. Having an extension where you set properties is often just a way to influence multiple places or tasks with one setting or to have a nicer end-user DSL, so indeed things you might not care about when automating. But it really depends on the concrete details from case to case.Ksawery Karwacki
09/05/2024, 10:42 AMpactPublish directly into the task. And the whole reason for the convention plugin is to not configure that manually for each repository so I do not care about extension availability at all.Vampire
09/05/2024, 11:01 AMPactPublish is not an extension. It is a data class that exists as property on the extension and as property on the task.Vampire
09/05/2024, 11:02 AMProperty and that is usually not that good an ideaVampire
09/05/2024, 11:03 AMProperty-basedKsawery Karwacki
09/05/2024, 11:12 AMPactPublish and all it's fields will be treated as regular types and not Properties and will not be lazy loaded. Even if it's not the case then still my instrumentation should be for whole PactPublish as specific fields are not lazy loaded so i cannot user Properties and Providers to configure them directly but I rather have to use map method or something like that to map to whole PactPublishVampire
09/05/2024, 12:01 PMProperty or the fields in those classes should be Property and individually wired to task inputs, depending on use-case.
But having a "plain" property which has "plain" properties is bad for lazy-configuration.