← Back to issue list

craftctl: update to comply with specification

View original Github issue

Metadata

Project
craft-parts
Number
#185
Type
pull request
State
merged
Author
cmatsuoka
Labels
Created
Updated
Closed

Current evaluation

Merged. Updated craftctl to comply with the specification, aligning default, set, and get commands with snapctl syntax, adding name validation, and enforcing consumption protection and single-part restrictions. Approved by reviewer.

Suggested action:

No scores available.

Issue body

The specification of craftctl lists the expected behavior of the `default`, `set`, and `get` commands, including syntax consistent with snapctl (when applicable), name validation, protection against set after consumption, and restricting variable setting to a single application-specified part. Update the existing implementation to follow the specification. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- CRAFT-935 CRAFT-75

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Updated craftctl to comply with the specification, aligning default, set, and get commands with snapctl syntax, adding name validation, and enforcing consumption protection and single-part restrictions. Approved by reviewer.
qwen3.6-35b-a3b-mtp-q6 Merged update aligning craftctl with the specification for default, set, and get commands. Includes snapctl-compatible syntax, name validation, consumption protection, and single-part variable restrictions. Approved after minor documentation feedback.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3751 craftctl: add snapcraftctl compatibility wrapper snapcraft merged Merged a snapcraftctl compatibility wrapper into craftctl to ease the core22 transition. The wrapper outputs an update message and includes spread tests. Approved by reviewer, passed CI, and merged into main.
80%
#1091 docs: revamp craftctl how-to craft-parts merged Merged documentation update revamping the craftctl how-to guide using upstream Snapcraft content. Approved by two reviewers, CI checks passed after fixes, and changelog updated for a minor release prior to merge.
78%
#6048 docs: correct 'craftctl' to 'snapctl' in config doc snapcraft merged Merged a documentation fix correcting craftctl to snapctl in the config guide. Approved by two reviewers with passing CI checks. The update modifies one file.
76%
#5738 docs: Service management with craftctl? snapcraft closed Documentation error acknowledged. Confirmed correct command is snapctl. Issue closed after contributor volunteered to submit a PR updating the text and clarifying craftctl versus snapctl usage.
73%
#728 parts: update craft-parts and enable craftctl charmcraft merged Merged update to craft-parts 1.4.0, enabling craftctl support. Users can now invoke craftctl default in part scriptlets to execute overridden step handlers. Approved and passed CI checks.
72%
#102 test: update craftctl test snapcraft-rocks merged Merged after two approvals and passing CI checks. Updates the spread test to validate the fix for snapcraft-rocks#33, modifying one file with seven added lines.
71%
#5743 docs: fix service management to snapctl snapcraft merged Merged documentation update correcting service management references from craftctl to snapctl. Approved by three reviewers, passed CI, squashed, and merged. Resolves #5738.
70%