Add `emit.warning()`
Metadata
Current evaluation
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.
Resolution: closed resolved
Reason: PR #358 'feat(emitter): add warning method' was merged as commit 4736b39, defining Emitter.warning() in craft_cli/messages.py and covering examples.py and unit/integration tests. Maintainer comments confirm the bounty was assigned and paid, after which the issue was closed.
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
Add a new emitter command, `emit.warning()`
### Why it needs to get done
`emit.progress(permanent=True)` is overloaded. It is the only way to log warnings like deprecation notices.
#### Acceptance criteria
* Warning method on the emitter that outputs (permanently) to stderr, includes the warning in the log, and provides a customisable prefix (Default: “WARNING: ”)
* Tests for this method
* At least one example in [examples.py](http://examples.py) that uses progress with permanent=False, progress with permanent=True, and warning.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | 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. | |
| stealth/space-bunny-alpha | — | closed resolved | Added Emitter.warning() with permanent stderr output, a customizable WARNING prefix, logging, tests, and examples via merged PR #358 (4736b39). | |
| qwen/qwen3.6-35b-a3b | — | — | Contributor @nryanl implemented the emit.warning() method, meeting all acceptance criteria. The feature was merged into the main branch, and the associated $300 bounty was successfully paid out. | |
| qwen/qwen3.6-35b-a3b | — | — | Assigned to nryanl, the emit.warning() method was implemented with tests and examples. The solution was merged, and the $300 bounty was successfully paid out. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged emit.warning() to replace overloaded emit.progress(permanent=True). Adds permanent stderr output, logging, customizable prefixes, tests, and examples. Contributor nryanl completed the implementation and received the bounty payout. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-cli#358
(confidence 95%)
Merged PR adding the warning method (commit 4736b39).
-
Related To:
canonical/craft-cli#362
(confidence 60%)
Later commit5396c87 'feat(dispatcher): add a warning for the default command' builds on the new emitter API.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #354 feat: Add emit.warning() | craft-cli | closed | Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #256. | |
| #355 feat: Add emit.warning() | craft-cli | closed | Closed unmerged and superseded by #358, which added Emitter.warning() in commit 4736b39 and resolved #256. |