Skip to content

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Nov 5, 2025

What I did
Moved progress package to cmd as a command line component and renamed
Removed uses of fmt.Printxx in pkg/compose to fully delegate UI rendering to display component

Related issue
#13230

(not mandatory) A picture of a cute animal, if possible in relation to what you did

@ndeloof ndeloof requested a review from a team as a code owner November 5, 2025 16:42
@ndeloof ndeloof requested a review from glours November 5, 2025 16:42
@codecov
Copy link

codecov bot commented Nov 5, 2025

@Sainan
Copy link

Sainan commented Nov 10, 2025

(not mandatory) A picture of a cute animal, if possible in relation to what you did

Don't worry, I got you covered

computer cat

Copy link
Contributor

@glours glours left a comment

Choose a reason for hiding this comment

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

Sounds good to me, and I didn't notice issue during my tests (build, up, CTRL+C, watch...)

@glours glours enabled auto-merge (rebase) November 12, 2025 08:23
@glours glours merged commit 45def51 into docker:main Nov 12, 2025
45 of 47 checks passed
@ndeloof ndeloof deleted the display branch November 12, 2025 10:36
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.

3 participants