improvement(tasks): minor logging improvements

garden-io · Apr 29, 2019 · 18e0485 · 18e0485
1 parent 81b8d58
commit 18e0485
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/garden-service/src/tasks/build.ts b/garden-service/src/tasks/build.ts
@@ -69,7 +69,7 @@ export class BuildTask extends BaseTask {
 
     const log = this.log.info({
       section: this.getName(),
-      msg: `Syncing sources...`,
+      msg: `Staging build...`,
       status: "active",
     })
 

diff --git a/garden-service/src/tasks/push.ts b/garden-service/src/tasks/push.ts
@@ -105,7 +105,9 @@ export class PushTask extends BaseTask {
       throw err
     }
 
-    if (result.message) {
+    if (result.pushed) {
+      log.setSuccess({ msg: chalk.green(result.message || `Done (took ${log.getDuration(1)} sec)`), append: true })
+    } else if (result.message) {
       log.setWarn({ msg: result.message, append: true })
     } else {
       log.setSuccess({ msg: chalk.green(result.message || `Done (took ${log.getDuration(1)} sec)`), append: true })

diff --git a/garden-service/src/util/validate-dependencies.ts b/garden-service/src/util/validate-dependencies.ts
@@ -88,7 +88,11 @@ export function detectMissingDependencies(
     const errMsg = "Unknown dependencies detected.\n\n" +
       indentString(missingDepDescriptions.join("\n\n"), 2) + "\n"
 
-    return new ConfigurationError(errMsg, { "unknown-dependencies": missingDepDescriptions })
+    return new ConfigurationError(errMsg, {
+      unknownDependencies: missingDepDescriptions,
+      availableModules: Array.from(moduleNames),
+      availableServicesAndTasks: Array.from(runtimeNames),
+    })
   } else {
     return null
   }