Skip to content

language: consolidate language-specific coalescers#1940

Open
BradLugo wants to merge 1 commit into
quay:mainfrom
BradLugo:blugo/refactor-language-coalescers
Open

language: consolidate language-specific coalescers#1940
BradLugo wants to merge 1 commit into
quay:mainfrom
BradLugo:blugo/refactor-language-coalescers

Conversation

@BradLugo

@BradLugo BradLugo commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

All language ecosystem coalescers shared the same logic: skip layers without repos, associate packages with repos. The gobin coalescer had unnecessary repo name/PackageDB prefix filtering since scanners already ensure only relevant packages appear.

Reimplements a portion of #1275, which was reverted in #1562.

@BradLugo
BradLugo requested review from a team as code owners July 6, 2026 18:24
@BradLugo
BradLugo requested a review from hdonnay July 6, 2026 18:24
@BradLugo

BradLugo commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Note for reviewers: I bumped the gobin detectorVersion since that coalescer had a fundamental logic change. I can be easily persuaded not to bump any of the detector/indexer versions or to bump all of them (though I plan to bump all of them in the next language coalescer PR anyway).

@BradLugo
BradLugo force-pushed the blugo/refactor-language-coalescers branch from 0827034 to 87432ad Compare July 6, 2026 18:29
All language ecosystem coalescers shared the same logic: skip layers
without repos, associate packages with repos. The gobin coalescer had
unnecessary repo name/PackageDB prefix filtering since scanners already
ensure only relevant packages appear.

Signed-off-by: Brad Lugo <blugo@redhat.com>
@BradLugo
BradLugo force-pushed the blugo/refactor-language-coalescers branch from 87432ad to f392484 Compare July 6, 2026 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant