← Back to issue list

extensions: add cli commands

View original Github issue

Metadata

Project
rockcraft
Number
#292
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged addition of list-extensions and expand-extensions CLI commands matching Snapcraft semantics, plus test refactoring. Resolves #274. Approved by two reviewers, passed CI, and modified 15 files (+452/-109 lines).

Suggested action:

No scores available.

Issue body

The commands are 'list-extensions' and 'expand-extensions' and they have the same semantics as the Snapcraft equivalents. Some minor refactoring to re-use testing data accross multiple tests. Fixes #274 - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged addition of list-extensions and expand-extensions CLI commands matching Snapcraft semantics, plus test refactoring. Resolves #274. Approved by two reviewers, passed CI, and modified 15 files (+452/-109 lines).
qwen3.6-35b-a3b-mtp-q6 Merged addition of list-extensions and expand-extensions CLI commands matching Snapcraft semantics, with minor test refactoring. Resolves issue #274.
qwen3.6-35b-a3b-mtp-q6 Merged. Adds list-extensions and expand-extensions CLI commands mirroring Snapcraft semantics. Includes test data refactoring and resolves #274.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1268 feat(extensions): add cli commands charmcraft merged Merged. Introduces CLI commands expand-extensions and list-extensions to resolve issue #1242. Approved by two reviewers, the change modifies ten files with 344 additions and 41 deletions.
82%
#3679 Expand extensions snapcraft merged Merged into snapcraft/7.0 after approval and passing CI checks. The changes expand extensions, address CRAFT-948, and increased overall coverage by 0.17%.
79%
#4110 extensions: parse snapcraft.yaml before expanding extensions snapcraft merged Merged fix for the expand-extensions command to parse snapcraft.yaml before expanding extensions. Resolves core22 parsing issues with parse-info and advanced grammar keywords. Approved, passed CI, and merged into main.
79%
#3678 commands: list-extensions and extensions alias snapcraft merged Merged into snapcraft/7.0. Implements the list-extensions command and extensions alias, integrating new codebase extensions with legacy ones via inheritance. Approved by reviewers and passed CI checks.
79%
#2349 extensions: use extension docstring snapcraft merged Merged into master. Replaces the generic snapcraft extension description with extension-specific docstrings, resolving bug LP: #1797655. Approved by reviewer and passed all CI checks.
76%
#2237 cli: add list-extensions command snapcraft merged Merged into master, adding the list-extensions CLI command and bash-completion for both list-extensions and extensions. Resolves Launchpad bug 1788639 with 100% test coverage and maintainer approval.
76%
#885 fix: Removed reference to snapcraft.yaml in expand extensions rockcraft open One-line fix removing the snapcraft.yaml reference from the expand-extensions help text and docstring, fixing #658. Approved by 2 reviewers with all CI checks green, but unmerged and inactive ~6 months; the fix is still unapplied in the codebase.
75%
#3266 cli: add --enable-experimental-extensions option for expand-extensions snapcraft merged Merged. Added --enable-experimental-extensions CLI flag to expand-extensions. Approved by one reviewer, passed CI checks, and committed with 13 additions and 3 deletions across one file.
75%
#2348 extensions: remove root extensions snapcraft merged Merged removal of root extension declarations from snapcraft.yaml. Resolves LP #1797631 by deferring application scope decisions until usage patterns clarify. Approved by maintainer, passed CI, and modified two files.
74%
#2347 extensions: support adding root properties snapcraft merged Merged: extensions can now add YAML snippets to the root of snapcraft.yaml, resolving LP: #1794531. Approved by a reviewer, passed CI with 100% diff coverage, and modified three files.
74%