From d0c896a8edae0a78c3bce49a2da9a69ff9465125 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ey=C3=BE=C3=B3r=20Magn=C3=BAsson?= <eysi09@gmail.com>
Date: Fri, 8 Mar 2019 17:22:08 +0100
Subject: [PATCH] fix: make sure to log build task success

---
 garden-service/src/tasks/build.ts | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/garden-service/src/tasks/build.ts b/garden-service/src/tasks/build.ts
index d021ca743e..b6453444c5 100644
--- a/garden-service/src/tasks/build.ts
+++ b/garden-service/src/tasks/build.ts
@@ -72,6 +72,10 @@ export class BuildTask extends BaseTask {
       status: "active",
     })
 
+    const logSuccess = () => {
+      log.setSuccess({ msg: chalk.green(`Done (took ${log.getDuration(1)} sec)`), append: true })
+    }
+
     await this.garden.buildDir.syncFromSrc(this.module)
 
     if (!this.force) {
@@ -80,6 +84,7 @@ export class BuildTask extends BaseTask {
       if (status.ready) {
         // this is necessary in case other modules depend on files from this one
         await this.garden.buildDir.syncDependencyProducts(this.module)
+        logSuccess()
         return { fresh: false }
       }
     }
@@ -97,7 +102,7 @@ export class BuildTask extends BaseTask {
       throw err
     }
 
-    log.setSuccess({ msg: chalk.green(`Done (took ${log.getDuration(1)} sec)`), append: true })
+    logSuccess()
     return result
   }
 }