feat(legacy): add a warning for the architecture shorthand on core20
Metadata
Current evaluation
Merged. Adds a warning for the core20 architecture shorthand to prevent confusion with core22 and core24 behavior. Approved by two reviewers, passed CI checks, and closed issue #5789.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/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 `make test`?
---
Adds a warning message when using the core20 architecture shorthand. This is because it behaves unexpectedly when compared to core22/core24, which causes confusion. See https://github.com/canonical/snapcraft/issues/5789#issuecomment-3386621445 for more context.
Closes #5789.
SNAPCRAFT-1227.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds a warning for the core20 architecture shorthand to prevent confusion with core22 and core24 behavior. Approved by two reviewers, passed CI checks, and closed issue #5789. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change adding a warning for the core20 architecture shorthand to clarify unexpected behavior compared to core22/core24. Closes #5789. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to add a warning for the core20 architecture shorthand, preventing unexpected behavior and confusion with core22/core24. Resolves #5789 and addresses SNAPCRAFT-1227. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3410 project: enable experimental target-arch support for core20 | snapcraft | merged | Merged PR enabling experimental target-arch support for core20 with an explicit warning. Approved by a maintainer, the change adds the flag to two files without affecting code coverage, and was integrated despite a noted test failure. | |
| #4436 docs(architectures): remove shorthand notation for core22 architectures | snapcraft | merged | Merged documentation update removing shorthand architecture notation for core22 snaps. The change discourages the notation as Launchpad lacks support, avoiding disruptive code removal. Approved by two reviewers and passed CI. |