Skip to content

Allow printed output to flush to prevent a race between progress updates and other output#23

Merged
adityaramani merged 9 commits intomainfrom
users/dkovba/progress-stdout
Jun 6, 2025
Merged

Allow printed output to flush to prevent a race between progress updates and other output#23
adityaramani merged 9 commits intomainfrom
users/dkovba/progress-stdout

Conversation

@dkovba
Copy link
Contributor

@dkovba dkovba commented Jun 6, 2025

This PR also improves clearing the last progress update after moving to a new task using setDescription().

@dkovba dkovba requested a review from adityaramani June 6, 2025 00:20
@dkovba dkovba marked this pull request as draft June 6, 2025 00:29
@dkovba dkovba force-pushed the users/dkovba/progress-stdout branch from ec901a3 to e40398c Compare June 6, 2025 02:41
@dkovba dkovba marked this pull request as ready for review June 6, 2025 02:42
@dkovba dkovba changed the title Move progress updates to stdout to prevent a race between progress updates and other output Allow printed output to flush to prevent a race between progress updates and other output Jun 6, 2025
@adityaramani
Copy link
Contributor

Thank you for the fixes!

@dkovba dkovba requested a review from adityaramani June 6, 2025 05:54
@dkovba dkovba requested a review from crosbymichael June 6, 2025 16:51
@adityaramani adityaramani merged commit 8ad99ef into main Jun 6, 2025
2 checks passed
@adityaramani adityaramani deleted the users/dkovba/progress-stdout branch June 6, 2025 18:48
owenarthur pushed a commit to owenarthur/container that referenced this pull request Jun 10, 2025
…tes and other output (apple#23)

This PR also improves clearing the last progress update after moving to
a new task using `setDescription()`.
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