chore: avoid caching entire node_modules

chadlwilson · chadlwilson · commit 9d76784ea885 · 2026-03-18T17:49:25.000+08:00
Can't recall why this wasn't done before, or whether it got accidentally removed in some refactoring....
diff --git a/buildSrc/src/main/groovy/com/thoughtworks/go/build/YarnInstallTask.groovy b/buildSrc/src/main/groovy/com/thoughtworks/go/build/YarnInstallTask.groovy
@@ -36,6 +36,7 @@ class YarnInstallTask extends DefaultTask {
   YarnInstallTask(ExecOperations execOperations) {
     this.execOperations = execOperations
     inputs.property('os', OperatingSystem.current().toString())
+    outputs.cacheIf { false }
   }
 
   @Input // not an @InputFile/InputDirectory, because we don't care about the contents of the workingDir itself

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ class YarnInstallTask extends DefaultTask {`
`36`	`36`	`YarnInstallTask(ExecOperations execOperations) {`
`37`	`37`	`this.execOperations = execOperations`
`38`	`38`	`inputs.property('os', OperatingSystem.current().toString())`
	`39`	`+ outputs.cacheIf { false }`
`39`	`40`	`}`
`40`	`41`
`41`	`42`	`@Input // not an @InputFile/InputDirectory, because we don't care about the contents of the workingDir itself`