fix: stop searching for snapshot artefacts when the snapshot tools are skipped (it's a cloud build, there aren't any snapshot artefacts locally)

DimitarTachev · DimitarTachev · commit 2feaa428756c · 2020-01-24T17:24:36.000+02:00
diff --git a/lib/after-prepare.js b/lib/after-prepare.js
@@ -12,7 +12,12 @@ module.exports = function (hookArgs) {
 		release: hookArgs.prepareData.release
 	};
 
-	if (env.snapshot && shouldSnapshot(shouldSnapshotOptions)) {
+	if (env.snapshot &&
+		shouldSnapshot(shouldSnapshotOptions) &&
+		(!hookArgs.prepareData ||
+			!hookArgs.prepareData.nativePrepare ||
+			!hookArgs.prepareData.nativePrepare.skipNativePrepare)) {
+
 		installSnapshotArtefacts(hookArgs.prepareData.projectDir);
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,12 @@ module.exports = function (hookArgs) {`
`12`	`12`	`release: hookArgs.prepareData.release`
`13`	`13`	`};`
`14`	`14`
`15`		`- if (env.snapshot && shouldSnapshot(shouldSnapshotOptions)) {`
	`15`	`+ if (env.snapshot &&`
	`16`	`+ shouldSnapshot(shouldSnapshotOptions) &&`
	`17`	`+ (!hookArgs.prepareData \|\|`
	`18`	`+ !hookArgs.prepareData.nativePrepare \|\|`
	`19`	`+ !hookArgs.prepareData.nativePrepare.skipNativePrepare)) {`
	`20`	`+`
`16`	`21`	`installSnapshotArtefacts(hookArgs.prepareData.projectDir);`
`17`	`22`	`}`
`18`	`23`	`}`