← Back to issue list

meta: fix content slot to set content when using source key

View original Github issue

Metadata

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

Current evaluation

Merged in canonical/snapcraft#3227 (commit 8cdbe2692), fixing content slots to preserve the content value when source is specified and adding unit tests.

Resolution: closed resolved

Reason: PR #3227 was approved and merged as commit 8cdbe2692. It fixes generated content slots to include the content key when source is used and adds accompanying unit tests.

Evaluated by: mr-cal-bot

No scores available.

Issue body

If using the 'source' key, the content key was lost. Fix and add accompanying unit tests. 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
stealth/space-bunny-alpha — closed resolved Merged in canonical/snapcraft#3227 (commit 8cdbe2692), fixing content slots to preserve the content value when source is specified and adding unit tests.
stealth/space-bunny-alpha — closed resolved Merged in snapcore/snapcraft#3227 (commit 8cdbe269), preserving the content key when source is used and adding unit tests.
stealth/space-bunny-alpha — closed resolved Merged in PR #3227 via commit 8cdbe2692, preserving the content key when source is set and adding unit tests.
qwen/qwen3.6-35b-a3b — — Merged fix ensuring content slot correctly sets content when using the source key. Added unit tests. Approved and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged fix for a bug where the content key was lost when using the source key in the meta content slot. Includes accompanying unit tests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2934 meta: fix for missing content slot's 'content' property snapcraft merged Merged PR #2934 fixed content-slot metadata initialization so snapcraft.yaml content is retained, with added unit coverage; landed as commit 4644651c4.
85%
#3874 meta: fix content plug detection snapcraft merged Merged PR #3874 in commit b962783c1, allowing empty content plug/slot definitions and deferring type validation until snap packaging.
71%