← Back to issue list

Improved the _Progresser and how it's used (CRAFT-1139).

View original Github issue

Metadata

Project
craft-cli
Number
#96
Type
pull request
State
merged
Author
facundobatista
Labels
Created
Updated
Closed

Current evaluation

Merged _Progresser improvements enabling timestamp display, explicit start/end markers, and removal of redundant context messages. Added timestamp support to the Printer. Approved, passed CI, and modified 5 files (+132/-47 lines).

Suggested action:

No scores available.

Issue body

Details: - it shows messages with timestamps when needed - it marks the beginning and end of the progress in general (very useful for example for logs, where the *progress itself* is not recorded) - as that context info is responsibility of the class itself, there is no more a separate message before putting it to work. Also added support in the Printer to show the timestamp for the progress bar, if indicated.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged _Progresser improvements enabling timestamp display, explicit start/end markers, and removal of redundant context messages. Added timestamp support to the Printer. Approved, passed CI, and modified 5 files (+132/-47 lines).
qwen3.6-35b-a3b-mtp-q6 Merged improvements to _Progresser to display timestamped messages, mark progress start and end for logging, and remove redundant pre-work messages. Added timestamp support to the Printer for progress bars. Resolves CRAFT-1139.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#7 Added the 'progress' method to send progress messages (CRAFT-98). craft-cli merged Merged. Adds a progress method to send progress messages for CRAFT-98. Approved by two reviewers, the change spans four files with 210 additions and passed all CI checks.
75%