From 00150e0e6dcad4d6958ef82a0be78ed433a9950b Mon Sep 17 00:00:00 2001 From: Alan Agius Date: Mon, 13 Jul 2020 18:12:28 +0200 Subject: [PATCH] fix(@angular-devkit/build-angular): don't emit empty warnings Closes #18231 --- .../src/angular-cli-files/utilities/stats.ts | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/stats.ts b/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/stats.ts index 415ea7c5a24d..385923e5c3fe 100644 --- a/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/stats.ts +++ b/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/stats.ts @@ -100,10 +100,11 @@ export function statsWarningsToString(json: any, statsConfig: any) { } return rs('\n' + warnings - .map((warning: any) => `${warning}`) - .filter((warning: string) => !ERRONEOUS_WARNINGS.some((erroneous) => erroneous.test(warning))) - .map((warning: string) => y(`WARNING in ${warning}`)) - .join('\n\n')); + .filter(m => !!m) + .map((warning: any) => `${warning}`) + .filter((warning: string) => !ERRONEOUS_WARNINGS.some((erroneous) => erroneous.test(warning))) + .map((warning: string) => y(`WARNING in ${warning}`)) + .join('\n\n')); } export function statsErrorsToString(json: any, statsConfig: any) { @@ -115,7 +116,11 @@ export function statsErrorsToString(json: any, statsConfig: any) { errors.push(...json.children.map((c: any) => c.errors)); } - return rs('\n' + errors.map((error: any) => r(`ERROR in ${error}`)).join('\n')); + return rs('\n' + errors + .filter(m => !!m) + .map((error: any) => r(`ERROR in ${error}`)) + .join('\n\n') + ); } export function statsHasErrors(json: any): boolean {