feat(@ngtools/webpack): use STDERR to report warnings

hansl · hansl · commit 2a1e19c4e4dc · 2018-08-21T09:57:34.000-07:00
diff --git a/packages/ngtools/webpack/src/transformers/ast_helpers.ts b/packages/ngtools/webpack/src/transformers/ast_helpers.ts
@@ -91,7 +91,7 @@ export function transformTypescript(
 
   // Log diagnostics if emit wasn't successfull.
   if (emitSkipped) {
-    console.log(diagnostics);
+    console.error(diagnostics);
 
     return null;
   }
diff --git a/packages/ngtools/webpack/src/type_checker.ts b/packages/ngtools/webpack/src/type_checker.ts
@@ -128,7 +128,7 @@ export class TypeChecker {
 
       if (warnings.length > 0) {
         const message = formatDiagnostics(warnings);
-        console.log(terminal.bold(terminal.yellow('WARNING in ' + message)));
+        console.error(terminal.bold(terminal.yellow('WARNING in ' + message)));
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ export function transformTypescript(`
`91`	`91`
`92`	`92`	`// Log diagnostics if emit wasn't successfull.`
`93`	`93`	`if (emitSkipped) {`
`94`		`- console.log(diagnostics);`
	`94`	`+ console.error(diagnostics);`
`95`	`95`
`96`	`96`	`return null;`
`97`	`97`	`}`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ export class TypeChecker {`
`128`	`128`
`129`	`129`	`if (warnings.length > 0) {`
`130`	`130`	`const message = formatDiagnostics(warnings);`
`131`		`- console.log(terminal.bold(terminal.yellow('WARNING in ' + message)));`
	`131`	`+ console.error(terminal.bold(terminal.yellow('WARNING in ' + message)));`
`132`	`132`	`}`
`133`	`133`	`}`
`134`	`134`	`}`