← Back to issue list

feat(plugins): use @ channel separator

View original Github issue

Metadata

Project
imagecraft
Number
#345
Type
pull request
State
merged
Author
smethnani
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Changes expected channel separator from "/" to "@" with optional surrounding whitespace. Fixes #342 --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/imagecraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [x] I've added or updated any relevant documentation. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged plugin updates changing the channel separator from slash to @ with optional surrounding whitespace. Resolves #342. Code passed linting and tests.
qwen3.6-35b-a3b-mtp-q6 Merged a feature update that changes the expected channel separator from "/" to "@" with optional surrounding whitespace, resolving issue #342.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1575 feat(snaps): use '@' for separating stage-snap names from their channels. craft-parts merged 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.
70%