← Back to issue list

slots: allow lists

View original Github issue

Metadata

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

Current evaluation

Merged following review and CI checks. Enables lists and other data types in core22 slot sub-fields to match core20 behavior. Achieved 100% diff coverage. Full slot validation casting remains deferred.

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`? ----- Allow for other data types in slot sub-fields in core22, similar to core20 behavior. The TODO for slot validation (casting and validating ContentSlots, DbusSlots) is still left open, as this is a much larger effort. (CRAFT-1084)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged following review and CI checks. Enables lists and other data types in core22 slot sub-fields to match core20 behavior. Achieved 100% diff coverage. Full slot validation casting remains deferred.
qwen3.6-35b-a3b-mtp-q6 Merged into main, enabling lists and other data types in core22 slot sub-fields to match core20 behavior. The change maintains 100% test coverage, while broader slot validation remains a future task.
qwen3.6-35b-a3b-mtp-q6 Merged into main. Enables lists in core22 slot sub-fields to match core20 behavior. Achieved 100% diff coverage with no overall impact. Comprehensive validation for ContentSlots and DbusSlots remains pending.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3755 meta: add slots snapcraft merged Merged to main following maintainer approval and passing CI checks. The change adds slots metadata across three files, achieving 100% diff coverage and resolving a reported issue.
70%