From 60be833ffd313039bbd3aa0155bb751f6971dcaf Mon Sep 17 00:00:00 2001 From: Federico Builes Date: Mon, 26 Sep 2022 12:01:39 +0200 Subject: [PATCH] Update manifest formatting in output. --- src/main.ts | 31 +++++++++++++++++++------------ 1 file changed, 19 insertions(+), 12 deletions(-) diff --git a/src/main.ts b/src/main.ts index 994e67f..ea307ae 100644 --- a/src/main.ts +++ b/src/main.ts @@ -137,17 +137,28 @@ function renderSeverity( return `${styles.color[color].open}(${severity} severity)${styles.color[color].close}` } -function renderChangeType( - change_type: 'added' | 'modified' | 'removed' -): string { +function renderScannedDependency(change: Change): string { + const changeType: string = change.change_type + + if (changeType !== 'added' && changeType !== 'removed') { + throw new Error(`Unexpected change type: ${changeType}`) + } + const color = ( { added: 'green', - modified: 'yellow', removed: 'red' } as const - )[change_type] - return `${styles.color[color].open}${change_type}${styles.color[color].close}` + )[changeType] + + const icon = ( + { + added: '+', + removed: '-' + } as const + )[changeType] + + return `${styles.color[color].open}${icon} ${change.manifest}@${change.version}${styles.color[color].close}` } function printLicensesError(changes: Change[]): void { @@ -190,13 +201,9 @@ function printScannedDependencies(changes: Change[]): void { for (const [manifestName, manifestChanges] of Object.entries( dependencies )) { - core.info(`${styles.bold.open}${manifestName}`) + core.info(`File: ${styles.bold.open}${manifestName}${styles.bold.close}`) for (const change of manifestChanges) { - core.info( - `${renderChangeType(change.change_type)}\t ${change.name}@${ - change.version - }` - ) + core.info(`${renderScannedDependency(change)}`) } } })