feat: improve lifecycle messages
Metadata
Current evaluation
Merged into main following approval from two reviewers and passing CI checks. The change improves lifecycle messages across six files (+61/-54 lines) with 87.50% diff coverage.
Suggested action: —
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint`?
- [ ] Have you successfully run `pytest tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into main following approval from two reviewers and passing CI checks. The change improves lifecycle messages across six files (+61/-54 lines) with 87.50% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main, improving lifecycle messages by adapting them from the craft application. The update achieved 87.50% diff coverage with zero overall coverage regression. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Updates lifecycle messages by adopting implementations from the craft application. The merge increased overall coverage by 0.00% with 87.50% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #62 fix: improve lifecycle messages | craft-application | merged | Merged into main after two approvals. Improved lifecycle messages and expanded human-readable text with all variances across three files. Minor coverage decrease was dismissed as a Codecov artifact. | |
| #4461 fix: improve lifecycle messages | snapcraft | merged | Merged after approval and passing CI. The change improves lifecycle messages by refactoring progress emission from open_stream to explicit progress calls, maintaining code coverage. | |
| #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. | |
| #4584 lifecycle commands | snapcraft | merged | Merged implementation of lifecycle commands. Approved by two reviewers, passed all CI checks, and modified 15 files with full test coverage. | |
| #98 some minor lifecycle papercuts | craft-application | merged | Merged into main after two approvals and passing CI. Adds a progress message during lifecycle initialization and exposes ProjectInfo for testing. The commit introduces 14 lines across two files with 100% diff coverage. | |
| #204 docs: fix typo in lifecycle manager docstrings | craft-parts | merged | Merged a documentation fix correcting a typo in the lifecycle manager docstrings. The single-line change to one file received reviewer approval and passed CI checks. | |
| #265 lifecycle_manager: minor primed packages sort refactoring | craft-parts | merged | Merged a minor refactoring to fix list sorting in lifecycle_manager after debug instrumentation was removed. Approved by one reviewer, passed CI checks, and applied with a +1/-3 line change. | |
| #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. | |
| #4452 test(spread): check for new lifecycle message | snapcraft | merged | Merged a test update to verify a new lifecycle message, resolving a regression from #4431. Approved by two reviewers with passing CI and stable coverage. The fix unblocks PR #4450. | |
| #29 commands: add basic lifecycle commands | craft-application | merged | Merged after two approvals and passing CI. Adds basic lifecycle commands across eight files, introducing 748 lines and removing 11. Maintains 100 percent test coverage. |