← Back to issue list

Add `emit.warning()`

View original Github issue

Metadata

Project
craft-cli
Number
#256
Type
issue
State
closed
Author
mr-cal
Labels
Bounty
Created
Updated
Closed

Current evaluation

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.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Added emit.warning() with customizable prefix and permanent stderr output. Assigned to @nryanl via a $300 bounty. A pull request was submitted, and the bounty payout confirms successful merge and acceptance.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#355 feat: Add emit.warning() craft-cli closed Adding emit.warning() was closed by the author and superseded by an alternative approach in issue #256. Changes were not merged.
73%