From 2d1d679f581b68161f4ee02852d1307ba60e84d8 Mon Sep 17 00:00:00 2001 From: Federico Builes Date: Mon, 26 Sep 2022 19:13:25 +0200 Subject: [PATCH] Move manifest grouping outside main.ts --- src/dependency-graph.ts | 17 +++++++++++++++++ src/main.ts | 14 ++------------ 2 files changed, 19 insertions(+), 12 deletions(-) diff --git a/src/dependency-graph.ts b/src/dependency-graph.ts index 7baf9c9..0d0902f 100644 --- a/src/dependency-graph.ts +++ b/src/dependency-graph.ts @@ -29,3 +29,20 @@ export async function compare({ ) return ChangesSchema.parse(changes) } + +export function groupDependenciesByManifest( + changes: Changes +): Map { + const dependencies: Map = new Map() + for (const change of changes) { + const manifestName = change.manifest + + if (dependencies.get(manifestName) === undefined) { + dependencies.set(manifestName, []) + } + + dependencies.get(manifestName)?.push(change) + } + + return dependencies +} diff --git a/src/main.ts b/src/main.ts index 57b413c..f5c3618 100644 --- a/src/main.ts +++ b/src/main.ts @@ -161,19 +161,9 @@ function renderScannedDependency(change: Change): string { return `${styles.color[color].open}${icon} ${change.manifest}@${change.version}${styles.color[color].close}` } -function printScannedDependencies(changes: Change[]): void { +function printScannedDependencies(changes: Changes): void { core.group('Dependency Changes', async () => { - // group changes by manifest - const dependencies: Map = new Map() - for (const change of changes) { - const manifestName = change.manifest - - if (dependencies.get(manifestName) === undefined) { - dependencies.set(manifestName, []) - } - - dependencies.get(manifestName)?.push(change) - } + const dependencies = dependencyGraph.groupDependenciesByManifest(changes) for (const manifestName of dependencies.keys()) { const manifestChanges = dependencies.get(manifestName) || []