Skip to content

Fix (reduce) evals summary width#2160

Merged
dgageot merged 1 commit intodocker:mainfrom
gtardif:fix_summary_width
Mar 18, 2026
Merged

Fix (reduce) evals summary width#2160
dgageot merged 1 commit intodocker:mainfrom
gtardif:fix_summary_width

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 18, 2026

reduce width dedicate to progress bar, and reduce failure details labels

[████░░░░░░]  48% (32/66) ✓7 ✗25 (relv ✗16, size ✗3, tools ✗6)

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif gtardif requested a review from a team as a code owner March 18, 2026 14:58
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The changes correctly implement the stated goal of reducing progress bar width to make room for failure details. No bugs found in the changed code.

The progress bar width reduction from 50 to 10 characters is intentional per the PR description, and the label abbreviations ("relv", "tools") are reasonable trade-offs for space constraints.

@dgageot dgageot merged commit ce11fc5 into docker:main Mar 18, 2026
8 checks passed
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