fix: clean ephemeral line before carriage return
Metadata
Current evaluation
Closed unmerged as superseded by PR #270, which fixed #263 in commit 0ca6940 by correctly printing multi-line errors.
Resolution: closed superseded
Reason: PR #265 was closed without merging; merged PR #270 replaced its approach and fixed the same error-output corruption in commit 0ca6940.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #263
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Closed unmerged as superseded by PR #270, which fixed #263 in commit 0ca6940 by correctly printing multi-line errors. | |
| stealth/space-bunny-alpha | — | closed superseded | Closed unmerged; #270 (commit 0ca6940) fixed #263 by correcting multi-line error output, superseding the proposed ephemeral-line cleanup. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged to resolve issue #263. The PR fixes ephemeral line handling before carriage returns, adding 33 lines across two files. Maintainer sergiusens authored the change, confirmed as necessary after a prior partial fix proved insufficient. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
canonical/craft-cli#270
(confidence 100%)
Merged replacement implementation for fixing issue #263.
-
Duplicate Of:
canonical/craft-cli#210
(confidence 92%)
Earlier partial fix that removed trailing spaces before carriage returns.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #241 Missing line feed in error output | craft-cli | closed | Closed as a duplicate of #263; its line-overwrite bug was fixed by merged PR #210 (c62ce3a) by removing trailing spaces in brief-mode output. |