Skip to content

progress: Move indicatif to CLI, introduce ProgressReporter API#295

Open
cgwalters wants to merge 1 commit into
composefs:mainfrom
cgwalters:issue-140-progress-api
Open

progress: Move indicatif to CLI, introduce ProgressReporter API#295
cgwalters wants to merge 1 commit into
composefs:mainfrom
cgwalters:issue-140-progress-api

Conversation

@cgwalters
Copy link
Copy Markdown
Collaborator

Library crates composefs-oci and composefs-http previously created indicatif progress bars directly, coupling them to terminal output.

But callers like bootc really need structured metadata so they can render their own progress.

Generated-by: OpenCode (Claude Sonnet 4.6)

@cgwalters cgwalters enabled auto-merge May 20, 2026 12:46
@cgwalters cgwalters added this to the 0.5 milestone May 20, 2026
Library crates composefs-oci and composefs-http previously created
indicatif progress bars directly, coupling them to terminal output.

But callers like bootc really need structured metadata so they
can render their own progress.

Closes: composefs#140

Generated-by: OpenCode (Claude Sonnet 4.6)
Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters cgwalters force-pushed the issue-140-progress-api branch from dbee396 to 3ed9d63 Compare May 22, 2026 17:33
Copy link
Copy Markdown
Collaborator

@Johan-Liebert1 Johan-Liebert1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cgwalters cgwalters added this pull request to the merge queue May 25, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 25, 2026
@Johan-Liebert1
Copy link
Copy Markdown
Collaborator

Debian tests are failing. I have a fix here #298

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