← Back to issue list

feat(emitter): add warning method

View original Github issue

Metadata

Project
craft-cli
Number
#358
Type
pull request
State
merged
Author
nryanl
Labels
—
Created
Updated
Closed

Current evaluation

Merged Emitter.warning() with stderr output, a configurable WARNING prefix, logging, tests, and examples in #358 (commit 4736b39), resolving #256.

Resolution: closed resolved

Reason: PR #358 was merged with all CI checks passing. Commit 4736b39 adds Emitter.warning() and closes canonical/craft-cli#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`? --- Adds a warning method to the Emitter class. Resolves #256.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged Emitter.warning() with stderr output, a configurable WARNING prefix, logging, tests, and examples in #358 (commit 4736b39), resolving #256.
stealth/space-bunny-alpha — closed resolved Merged Emitter.warning() in canonical/craft-cli#358 (commit 4736b39), resolving #256 after two approvals and all CI checks passed.
qwen/qwen3.6-35b-a3b — — Merged the addition of a warning method to the Emitter class, resolving issue #256. The change passed all CI checks, received two approvals, and incorporated reviewer feedback.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-cli#256 (confidence 100%)

    This issue requested Emitter.warning() and was resolved by the merged implementation in PR #358.

Related issues

Issue Project State Summary Similarity
#354 feat: Add emit.warning() craft-cli closed Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #256.
83%
#355 feat: Add emit.warning() craft-cli closed Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #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.
83%