Move manifest grouping outside main.ts

This commit is contained in:
Federico Builes
2022-09-26 19:13:25 +02:00
parent a3563a05bc
commit 2d1d679f58
2 changed files with 19 additions and 12 deletions

View File

@@ -29,3 +29,20 @@ export async function compare({
)
return ChangesSchema.parse(changes)
}
export function groupDependenciesByManifest(
changes: Changes
): Map<string, Changes> {
const dependencies: Map<string, Changes> = 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
}

View File

@@ -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<string, Change[]> = 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) || []