← Back to issue list

cli: merge build options into provider options

View original Github issue

Metadata

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

Current evaluation

Merged consolidation of build options into provider options in the CLI to simplify handling, as they are provider-specific. Approved by one reviewer, passed CI checks, and modified three files.

Suggested action:

No scores available.

Issue body

In reality these options only work with specific providers. Merge the build options and provider option lists to simplify the option handling code. Signed-off-by: Chris Patterson <chris.patterson@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged consolidation of build options into provider options in the CLI to simplify handling, as they are provider-specific. Approved by one reviewer, passed CI checks, and modified three files.
qwen3.6-35b-a3b-mtp-q6 Merged build options into provider options to simplify option handling, as these settings only apply to specific providers. The pull request was merged by maintainer cjp256 without additional comments or revisions.
qwen3.6-35b-a3b-mtp-q6 Merged build options into provider options to simplify CLI configuration handling. Consolidating these provider-specific settings streamlines option parsing. Approved and merged by maintainer cjp256.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2678 cli: introduce --provider snapcraft merged Merged into master. Adds a --provider CLI flag for host, lxd, and multipass, replacing legacy flags. Refactors option handling, cleans up configuration, and increases coverage. Approved and passed CI.
72%
#2275 build providers: use the provider if exported snapcraft merged Merged an update to build provider logic that enables using a provider if it is exported. Approved by a reviewer, passed CI checks, and integrated into the codebase.
71%