← Back to issue list

cli: only issue warning when checking for usage of sudo

View original Github issue

Metadata

Project
snapcraft
Number
#3240
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

Merged a maintainer pull request that updates the CLI to only issue a warning when checking for sudo usage. Approved by one reviewer, the change modifies four files and reduces the codebase by 38 lines.

Suggested action: —

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a maintainer pull request that updates the CLI to only issue a warning when checking for sudo usage. Approved by one reviewer, the change modifies four files and reduces the codebase by 38 lines.
qwen3.6-35b-a3b-mtp-q6 — — Merged CLI modifications to only issue warnings when checking for sudo usage. The maintainer integrated the change directly without additional discussion.
qwen3.6-35b-a3b-mtp-q6 — — Merged to the main branch, the CLI now only issues a warning when checking for sudo usage. The maintainer-authored update was integrated without additional discussion.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3275 cli: ignore sudo warning when using multipass snapcraft merged Merged a pull request that updates the CLI to ignore sudo warnings when using multipass. The change (+8/-2 lines in one file) was approved by one reviewer, passed CI checks, and successfully integrated into the main branch.
78%
#3150 cli: error/warn when using sudo snapcraft merged Merged PR #3150 into master. Adds CLI checks to error on sudo for LXD, Multipass, and remote-build, while warning host and destructive-mode users. Updates spread tests to prevent permission errors.
75%
#3159 cli: improve sudo detection with euid check for warnings/errors snapcraft merged Merged improvements to sudo detection that verify the effective user ID, preventing unnecessary warnings when switching users. Approved by a maintainer, passed CI, and updated 4 files.
71%