fix: use progress for the packing messages
Metadata
Current evaluation
Merged fix replacing the appending Packing message with a progress indicator that overwrites previous output. Approved by two reviewers, passed CI, and achieved 100% diff coverage across seven changed files.
Suggested action: —
No scores available.
Issue body
Replace the previous "Packing..." message instead of appending to it.
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix replacing the appending Packing message with a progress indicator that overwrites previous output. Approved by two reviewers, passed CI, and achieved 100% diff coverage across seven changed files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Replaced the appending Packing message with a progress update mechanism. Achieved 100% diff coverage with no overall coverage impact. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #52 lifecycle: emit progress for packing | craft-application | merged | Merged into main. Adds progress emission during packing to prevent a long ticker on the Lifecycle finished message. Approved by two reviewers, passed CI, and modified two files with two added lines. | |
| #121 fix: don't duplicate lifecycle progress messages | craft-application | merged | Merged into main following two approvals and passing CI. The change prevents duplicate lifecycle progress messages across three files. Code coverage remains at 99.01% with 100% diff coverage. | |
| #829 fix(pack): put a gate around skipping repack. | craft-application | merged | Merged a fix that adds a gate around skipping repack. Approved by three reviewers and passed all CI checks. The change modifies five files with 21 additions. |