← Back to issue list

feat: Add emit.warning()

View original Github issue

Metadata

Project
craft-cli
Number
#354
Type
pull request
State
closed
Author
Mr-Sunglasses
Labels
—
Created
Updated
Closed

Current evaluation

Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #256.

Resolution: closed superseded

Reason: PR #354 was replaced by canonical/craft-cli#358. That PR merged Emitter.warning() in commit 4736b39 and resolved issue #256.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you followed the guidelines for contributing? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `make lint && make test`? fix: #256 ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #256.
stealth/space-bunny-alpha — closed superseded Closed unmerged and superseded by canonical/craft-cli#358, which implemented Emitter.warning() in commit 4736b39 and resolved #256.
qwen/qwen3.6-35b-a3b — — The emit.warning() feature request was closed without merging. It contained zero code changes, received no reviews or comments, and remained inactive for over a year, indicating it was abandoned.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/craft-cli#358 (confidence 100%)

    Merged replacement implementation for warning output, including tests and documentation.

  • Likely Fixed By: canonical/craft-cli#256 (confidence 99%)

    Original feature request, resolved by PR #358 and commit 4736b39.

Related issues

Issue Project State Summary Similarity
#355 feat: Add emit.warning() craft-cli closed Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #256.
100%
#358 feat(emitter): add warning method craft-cli merged Merged Emitter.warning() with stderr output, a configurable WARNING prefix, logging, tests, and examples in #358 (commit 4736b39), resolving #256.
83%
#256 Add `emit.warning()` craft-cli closed emit.warning() was added with permanent stderr output, customisable prefix, logging, tests and examples by merged PR #358 (commit 4736b39); bounty paid to nryanl and the issue closed.
76%