Skip to content

Export BuildFileIndex; decouple crawl from progress bar - #66

Merged
fproulx-boostsecurity merged 1 commit into
mainfrom
maint/export-buildindex
Jul 21, 2026
Merged

Export BuildFileIndex; decouple crawl from progress bar#66
fproulx-boostsecurity merged 1 commit into
mainfrom
maint/export-buildindex

Conversation

@SUSTAPLE117

Copy link
Copy Markdown
Contributor

Extract collector.buildFileIndex into an exported free function collector.BuildFileIndex

Progress is abstracted behind a new pkg/progress.Reporter interface (Start/Update/Done) with a NoOp default, replacing the hardcoded progressbar. BuildFileIndex depends only on the interface

@SUSTAPLE117
SUSTAPLE117 force-pushed the maint/export-buildindex branch from b0d4ad0 to 65cebc8 Compare July 13, 2026 14:57
auto-merge was automatically disabled July 21, 2026 19:34

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 19:34

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 19:35

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:02

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:06

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:15

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:15

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:15

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:16

Pull Request is not mergeable

auto-merge was automatically disabled July 21, 2026 20:16

Pull Request is not mergeable

Base automatically changed from maint/export-dedup to main July 21, 2026 20:19
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@fproulx-boostsecurity
fproulx-boostsecurity merged commit 03f5c18 into main Jul 21, 2026
5 checks passed
@fproulx-boostsecurity
fproulx-boostsecurity deleted the maint/export-buildindex branch July 21, 2026 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants