Merge pull request #64 from bitrise-io/fix-error-log

fix: Return error if xcode commands fail
bitrise-io · Nov 25, 2024 · 9dba0fa · 9dba0fa
2 parents ee5eb84 + 40639bb
commit 9dba0fa
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/cmd/restoreXcodeDerivedDataFiles.go b/cmd/restoreXcodeDerivedDataFiles.go
@@ -73,7 +73,7 @@ var restoreXcodeDerivedDataFilesCmd = &cobra.Command{
 			}
 		}
 
-		tracker.LogRestoreFinished(time.Since(startT), err)
+		tracker.LogRestoreFinished(time.Since(startT), cmdError)
 		if cmdError != nil {
 			return fmt.Errorf("restore Xcode DerivedData from Bitrise Build Cache: %w", cmdError)
 		}

diff --git a/cmd/saveXcodeDerivedDataFiles.go b/cmd/saveXcodeDerivedDataFiles.go
@@ -76,9 +76,9 @@ var saveXcodeDerivedDataFilesCmd = &cobra.Command{
 			}
 		}
 
-		tracker.LogSaveFinished(time.Since(startT), err)
-		if err != nil {
-			return fmt.Errorf("save Xcode cache into Bitrise Build Cache: %w", err)
+		tracker.LogSaveFinished(time.Since(startT), cmdError)
+		if cmdError != nil {
+			return fmt.Errorf("save Xcode cache into Bitrise Build Cache: %w", cmdError)
 		}
 
 		logger.TInfof("✅ Cache directories saved into Bitrise Build Cache ")