Wrap non-publish case in else block to clarify function flow

Signed-off-by: Zander Mackie <zmackie@gmail.com>
mdelillo · May 5, 2020 · 6907768 · 6907768
1 parent 00a01ca
commit 6907768
Showing 1 changed file with 14 additions and 14 deletions.
diff --git a/internal/build/phases.go b/internal/build/phases.go
@@ -187,22 +187,22 @@ func (l *Lifecycle) newExport(repoName, runImage string, publish bool, launchCac
 		)
 
 		return phaseFactory.New(configProvider), nil
-	}
-
-	args = append([]string{"-daemon", "-launch-cache", launchCacheDir}, args...)
-	binds = append(binds, fmt.Sprintf("%s:%s", launchCacheName, launchCacheDir))
+	} else {
+		args = append([]string{"-daemon", "-launch-cache", launchCacheDir}, args...)
+		binds = append(binds, fmt.Sprintf("%s:%s", launchCacheName, launchCacheDir))
 
-	configProvider := NewPhaseConfigProvider(
-		"exporter",
-		l,
-		WithDaemonAccess(),
-		WithArgs(
-			l.withLogLevel(args...)...,
-		),
-		WithBinds(binds...),
-	)
+		configProvider := NewPhaseConfigProvider(
+			"exporter",
+			l,
+			WithDaemonAccess(),
+			WithArgs(
+				l.withLogLevel(args...)...,
+			),
+			WithBinds(binds...),
+		)
 
-	return phaseFactory.New(configProvider), nil
+		return phaseFactory.New(configProvider), nil
+	}
 }
 
 func (l *Lifecycle) withLogLevel(args ...string) []string {