← Back to issue list

config: add snap config model

View original Github issue

Metadata

Project
snapcraft
Number
#3898
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged into main following reviewer approval and passing CI. Adds a snap config model to parse data set via snap set snapcraft key=value, mirroring charmcraft. The change modifies 11 files with 317 additions.

Suggested action: —

No scores available.

Issue body

- [X] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `make lint`? - [X] Have you successfully run `pytest tests/unit`? ----- Allows Snapcraft to parse data set via `snap set snapcraft <key>=<value>`. Follows same paradigm as charmcraft. (CRAFT-1321)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged into main following reviewer approval and passing CI. Adds a snap config model to parse data set via snap set snapcraft key=value, mirroring charmcraft. The change modifies 11 files with 317 additions.
qwen3.6-35b-a3b-mtp-q6 — — Merged PR #3898 by mr-cal introduces a snap config model for Snapcraft, enabling parsing of values set via snap set snapcraft key=value. The change follows the charmcraft paradigm and resolves CRAFT-1321.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Adds a snap config model enabling Snapcraft to parse settings via snap set snapcraft, aligning with charmcraft. Diff coverage reached 100%.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#239 feat(snap_config): add snap_config model and utilities craft-application merged Merged snap_config model and utilities into craft-application. Imported from snapcraft, updated to coding standards, and made generic. Approved by two reviewers, passed CI, and initiates a three-part series for build mode configuration.
79%
#3893 config: add snap config model snapcraft closed Closed by author. The snap config model was abandoned and superseded by a new implementation leveraging the snaphelpers library for better configuration handling.
77%
#2954 cli: enable config file for base snapcraft command snapcraft merged Merged a single-line update enabling config file parsing for the base snapcraft command. The change extends existing lifecycle configuration support to the main entry point, approved by maintainers and passing all CI checks.
73%
#50 test: test the snapcraft actions craft-actions merged Merged adding tests for snapcraft actions, mirroring charmcraft tests. Approved by two reviewers with all CI checks passing. Updates two files with 89 additions and 1 deletion.
70%
#2411 cli: snapcraft init with a base snapcraft merged Merged. Adds a syntax documentation link after snapcraft init execution. Approved by one reviewer, passed CI checks, modified five files, and achieved 100% diff coverage. Resolves LP #1803776.
70%