From dd39ce6af8977ee88e4f925b11f752787f2084ed Mon Sep 17 00:00:00 2001
From: Paul Gschwendtner <paulgschwendtner@gmail.com>
Date: Wed, 1 Mar 2017 19:58:18 +0100
Subject: [PATCH] build: make app tasks optional

* Now all related app-tasks like `scss`, `html`, `assets` or `ts` are optional.
* This way app-tasks like for the `e2e-app` don't need to run no-op tasks (and also it is more convenient)
---
 tools/gulp/task_helpers.ts | 3 ++-
 tools/gulp/tasks/e2e.ts    | 3 ---
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/tools/gulp/task_helpers.ts b/tools/gulp/task_helpers.ts
index 68f9f8e24b79..3f32afed07d8 100644
--- a/tools/gulp/task_helpers.ts
+++ b/tools/gulp/task_helpers.ts
@@ -135,7 +135,8 @@ export function cleanTask(glob: string) {
 /** Build an task that depends on all application build tasks. */
 export function buildAppTask(appName: string) {
   const buildTasks = ['vendor', 'ts', 'scss', 'assets']
-    .map(taskName => `:build:${appName}:${taskName}`);
+    .map(taskName => `:build:${appName}:${taskName}`)
+    .filter(taskName => gulp.hasTask(taskName));
 
   return (done: () => void) => {
     gulpRunSequence(
diff --git a/tools/gulp/tasks/e2e.ts b/tools/gulp/tasks/e2e.ts
index 46ee68e8415e..ccc1d0f4901c 100644
--- a/tools/gulp/tasks/e2e.ts
+++ b/tools/gulp/tasks/e2e.ts
@@ -25,9 +25,6 @@ task(':build:e2eapp:vendor', vendorTask());
 /** Builds e2e app ts to js. */
 task(':build:e2eapp:ts', tsBuildTask(appDir));
 
-/** No-op (needed by buildAppTask). */
-task(':build:e2eapp:scss', sassBuildTask(outDir, appDir));
-
 /** Copies e2e app assets (html, css) to build output. */
 task(':build:e2eapp:assets', copyTask(appDir, outDir));