Move validation of the rule Gradle property to execution time (#18)

petertrr · web-flow · commit f2a45bc7176d · 2025-01-27T17:36:29.000+01:00
diff --git a/gradle-modules/src/main/kotlin/org/sonarsource/cloudnative/gradle/RuleApiUpdate.kt b/gradle-modules/src/main/kotlin/org/sonarsource/cloudnative/gradle/RuleApiUpdate.kt
@@ -80,13 +80,19 @@ fun Project.registerRuleApiGenerateTask(
             buildList {
                 add("generate")
                 add("-rule")
-                add(rule.orNull ?: error("To generate rule rspec, please provide -Prule=SXXXX"))
+                add(rule.getOrElse(""))
                 if (branch.isPresent) {
                     add("-branch")
                     add(branch.get())
                 }
             }
         )
+
+        doFirst {
+            if (!rule.isPresent) {
+                error("To generate rule rspec, please provide -Prule=SXXXX")
+            }
+        }
     }
 
 private fun Project.registerRuleApiTask(

Original file line number	Diff line number	Diff line change
`@@ -80,13 +80,19 @@ fun Project.registerRuleApiGenerateTask(`
`80`	`80`	`buildList {`
`81`	`81`	`add("generate")`
`82`	`82`	`add("-rule")`
`83`		`- add(rule.orNull ?: error("To generate rule rspec, please provide -Prule=SXXXX"))`
	`83`	`+ add(rule.getOrElse(""))`
`84`	`84`	`if (branch.isPresent) {`
`85`	`85`	`add("-branch")`
`86`	`86`	`add(branch.get())`
`87`	`87`	`}`
`88`	`88`	`}`
`89`	`89`	`)`
	`90`	`+`
	`91`	`+ doFirst {`
	`92`	`+ if (!rule.isPresent) {`
	`93`	`+ error("To generate rule rspec, please provide -Prule=SXXXX")`
	`94`	`+ }`
	`95`	`+ }`
`90`	`96`	`}`
`91`	`97`
`92`	`98`	`private fun Project.registerRuleApiTask(`