diff --git a/src/main.ts b/src/main.ts index c567f34..d4f7cc6 100644 --- a/src/main.ts +++ b/src/main.ts @@ -53,7 +53,11 @@ async function run(): Promise { } ) - summary.addSummaryToSummary(addedChanges, invalidLicenseChanges) + summary.addSummaryToSummary( + config.vulnerability_check ? addedChanges : null, + config.license_check ? invalidLicenseChanges : null + ) + if (config.vulnerability_check) { summary.addChangeVulnerabilitiesToSummary(addedChanges, minSeverity) printVulnerabilitiesBlock(addedChanges, minSeverity) diff --git a/src/summary.ts b/src/summary.ts index 99058de..f2d2832 100644 --- a/src/summary.ts +++ b/src/summary.ts @@ -4,17 +4,23 @@ import {SummaryTableRow} from '@actions/core/lib/summary' import {groupDependenciesByManifest, getManifestsSet, renderUrl} from './utils' export function addSummaryToSummary( - addedPackages: Changes, - invalidLicenseChanges: Record + addedPackages: Changes | null, + invalidLicenseChanges: Record | null ): void { core.summary .addHeading('Dependency Review') .addRaw('We found:') .addList([ - `${addedPackages.length} vulnerable package(s)`, - `${invalidLicenseChanges.unresolved.length} package(s) with invalid SPDX license definitions`, - `${invalidLicenseChanges.forbidden.length} package(s) with incompatible licenses`, - `${invalidLicenseChanges.unlicensed.length} package(s) with unknown licenses.` + ...(addedPackages + ? [`${addedPackages.length} vulnerable package(s)`] + : []), + ...(invalidLicenseChanges + ? [ + `${invalidLicenseChanges.unresolved.length} package(s) with invalid SPDX license definitions`, + `${invalidLicenseChanges.forbidden.length} package(s) with incompatible licenses`, + `${invalidLicenseChanges.unlicensed.length} package(s) with unknown licenses.` + ] + : []) ]) }