← Back to issue list

feat(snaps): use '@' for separating stage-snap names from their channels.

View original Github issue

Metadata

Project
craft-parts
Number
#1575
Type
pull request
State
merged
Author
lengau
Labels
Status: Needs information
Created
Updated
Closed

Current evaluation

Merged after implementing the @ separator for stage-snap names and channels. Resolved merge conflicts, updated docs and tests, and added whitespace validation. Approved by four reviewers with passing CI checks.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? - [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- Fixes #1574

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after implementing the @ separator for stage-snap names and channels. Resolved merge conflicts, updated docs and tests, and added whitespace validation. Approved by four reviewers with passing CI checks.
qwen/qwen3.6-35b-a3b Merged after resolving conflicts and updating the implementation to enforce strict @ separator usage without surrounding whitespace. Replaces the legacy / format for stage-snap names and channels, with added validation to reject spaces. Approved and integrated.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 85
needs review Updates snap channel parsing to use '@' separator instead of '/'. Approved by 4 reviewers, CI passing, maintainer ready to merge pending final review on whitespace validation placement.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 35
Confidence: 90
needs review Implements '@' separator for snap stage/build names, replacing legacy '/' format. Approved by 4 reviewers, CI passing, maintainer requests re-review to merge before handling deb support separately.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 15
Confidence: 90
needs review Updates snap channel parsing to use '@' separator instead of '/'. Maintainer requested re-review after resolving conflicts, fixing whitespace handling, and adding validation. CI passes.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 40
Confidence: 90
needs review Introduces '@' separator for stage-snap names and channels. Actively maintained and ready for re-review per maintainer request. CI shows pending checks and one lint failure.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 20
Confidence: 90
needs review Updates snap channel parsing to use '@' separator. Maintainer recently requested re-review after resolving conflicts and fixing whitespace, but 5 review comments remain and CI shows a failing lint check.
qwen/qwen3.6-35b-a3b
Staleness: 10
Complexity: 30
Confidence: 85
needs review Implements @ separator for stage-snap names and channels, resolves merge conflicts, and enforces strict whitespace rules. Awaiting re-review from maintainers after recent updates.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 85
needs review Updates snap channel separator from / to @ in stage-snap names. Currently under re-review with 8 unresolved comments and failing CI checks, but a maintainer has requested a merge pending final approval.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 20
Confidence: 90
needs review Updates snap channel separator from / to @, resolves conflicts and review feedback, and fixes lint. Currently awaiting re-review after maintainer indicated intent to merge it separately from deb support.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 35
Confidence: 90
needs review Updates snap channel parsing to use '@' separator. Resolved merge conflicts and enforced strict whitespace rules per reviewer feedback. Awaiting maintainer re-review and approval to merge.
qwen/qwen3.6-35b-a3b
Staleness: 10
Complexity: 20
Confidence: 85
needs review Introduces '@' separator for stage-snap names and channels. Resolves merge conflicts and addresses review feedback on whitespace handling. Currently awaiting maintainer re-review with minor CI failures on lint and minimum dependencies.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 35
Confidence: 95
needs review Updates snap channel parsing to use '@' separator instead of '/'. Awaiting maintainer re-review after merge conflict resolution and whitespace fixes. CI lint check is failing, one unresolved review comment remains.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 25
Confidence: 90
needs review Updates snap channel parsing to use '@' separator. Actively under review with recent maintainer feedback, unresolved comments, and failing lint checks.
qwen/qwen3.6-35b-a3b
Staleness: 10
Complexity: 25
Confidence: 90
keep open Updates snap channel parsing to use '@' separator. Currently under review with one unresolved comment, actively maintained, and explicitly blocked pending approval of an external specification.
qwen/qwen3.6-35b-a3b
Staleness: 20
Complexity: 25
Confidence: 85
keep open Updates snap channel separator syntax from / or = to @ with optional whitespace. Currently under review with changes requested, blocked pending a formal spec approval, and merge conflicts were recently resolved.
qwen/qwen3.6-35b-a3b
Staleness: 45
Complexity: 45
Confidence: 90
keep open Changes snap channel separator syntax from / or = to @ with optional whitespace. Currently under review with changes requested, blocked pending approval of an external specification for the new syntax.
qwen3.6-35b-a3b-mtp-q6
Staleness: 20
Complexity: 40
Confidence: 90
keep open Implements '@' separator for stage-snap names and channels, replacing '='. Currently paused pending approval of a new syntax specification by the team.
qwen3.6-35b-a3b-mtp-q6
Staleness: 15
Complexity: 30
Confidence: 85
needs review Updates snap channel separators to use at-sign with optional whitespace, updating tests and docs while retaining legacy syntax for compatibility. Currently under review, with maintainers discussing whether to add a runtime deprecation warning for the old format.

Update history

Date Change
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated
updated

Related issues

Issue Project State Summary Similarity
#2468 many: support for stage-snaps snapcraft merged Merged implementation of stage-snap support featuring advanced grammar and snap source extraction. Approved by reviewer, passed CI, and resolved Launchpad bug #1805214 across 15 files.
76%
#345 feat(plugins): use @ channel separator imagecraft merged Merged after two approvals and passing CI checks. Replaces the expected channel separator from slash to at-sign with optional surrounding whitespace, resolving issue #342 across seven files.
70%