← Back to issue list

utils: introduce confirm_with_user() helper for prompting (CRAFT-336)

View original Github issue

Metadata

Project
charmcraft
Number
#422
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged #422 via 802370cb, adding confirm_with_user() for consistent yes/no prompts with non-TTY defaults and managed-mode safeguards.

Resolution: closed resolved

Reason: PR #422 was approved and merged into main as commit 802370cb, implementing CRAFT-336 with the new interactive confirmation helper.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Chris Patterson <chris.patterson@canonical.com>

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #422 via 802370cb, adding confirm_with_user() for consistent yes/no prompts with non-TTY defaults and managed-mode safeguards.
stealth/space-bunny-alpha — closed resolved Merged #422 via 802370cb, adding confirm_with_user() for interactive prompts and implementing CRAFT-336.
stealth/space-bunny-alpha — closed resolved Merged #422 via commit 802370cb, adding a reusable interactive yes/no confirmation helper and implementing CRAFT-336.
stealth/space-bunny-alpha — closed resolved PR #422 by cjp256 merged as commit 802370cb, adding utils.confirm_with_user() (charmcraft/utils/cli.py) for yes/no prompts, returning defaults without a tty, pausing the emitter, and raising in managed-mode; CRAFT-336.
stealth/space-bunny-alpha — closed resolved Merged #422 via 802370cb, adding confirm_with_user() for interactive prompts and implementing CRAFT-336.
stealth/space-bunny-alpha — closed resolved Merged canonical/charmcraft#422, adding a shared confirm_with_user() prompting helper across two files to centralize user confirmation handling for CRAFT-336.
qwen/qwen3.6-35b-a3b — — Merged introduces confirm_with_user() helper for user prompts in utils. Approved by two reviewers, the change adds 104 lines across two files and resolves CRAFT-336.
qwen3.6-35b-a3b-mtp-q6 — — Merged, introducing a confirm_with_user() helper in utils for interactive prompts. Authored by maintainer cjp256 and integrated without additional comments.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3938 providers: move `confirm_with_user()` call to providers.py (#91) snapcraft merged Merged #3938 moved Snapcraft's confirm_with_user() prompt out of the craft-providers interface and into providers.py; landed as commit 792d45f2f.
70%