Skip to content

Commit 0ce3863

Browse files
committed
Trim verbose comment in output-manifest-dynamic-sbom-inference
1 parent 53502d0 commit 0ce3863

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

src/commands/manifest/output-manifest-dynamic-sbom-inference.mts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@ import type { RecursiveManifestOutcome } from './generate-recursive-manifests.mt
77
import type { CResult, OutputKind } from '../../types.mts'
88

99
function renderTable(outcomes: readonly RecursiveManifestOutcome[]): string {
10-
// A reactor member covered by its parent's own facts run is implied by that
11-
// parent's line already showing up above it; listing it again here is just
12-
// noise, and the aggregate count still shows up in summarize().
10+
// A covered reactor member is implied by its parent's line above it.
1311
return outcomes
1412
.filter(o => o.status !== 'skippedCovered')
1513
.map(
@@ -19,11 +17,6 @@ function renderTable(outcomes: readonly RecursiveManifestOutcome[]): string {
1917
.join('\n')
2018
}
2119

22-
// Only the generated count is reported: a failure aborts the whole walk
23-
// immediately (already reported via its own fail message) rather than
24-
// accumulating alongside successes, and the disabled/covered/empty buckets
25-
// count candidate directories, not independent build roots, so a total
26-
// there would be just as misleading as the removed "N build root(s)" one.
2720
function summarize(outcomes: readonly RecursiveManifestOutcome[]): string {
2821
const generated = outcomes.filter(o => o.status === 'generated').length
2922
return `Generated ${generated} Socket facts file(s).`

0 commit comments

Comments
 (0)