fix: generate stricter default platforms
Metadata
Current evaluation
Merged fix for issue #52 that generates stricter default platforms equivalent to Debian's any architecture value. Approved by two reviewers, passed all CI checks, and modified 10 files.
Suggested action: —
No scores available.
Issue body
Fixes #52
This produces a default platform set that is equivalent to Debian's `any` Architecture value. https://www.debian.org/doc/debian-policy/ch-controlfields.html#architecture
- [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`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for issue #52 that generates stricter default platforms equivalent to Debian's any architecture value. Approved by two reviewers, passed all CI checks, and modified 10 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after passing lint and tests. Resolves issue #52 by generating stricter default platforms equivalent to Debian's any architecture value. Successfully integrated by maintainer lengau. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix updating default platform generation to match Debian's any architecture, resolving issue #52. The change passed linting and tests, though a reviewer noted it may have broken Copilot functionality. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #442 fix(platform): automatically vectorise architectures | craft-application | merged | Merged fix allowing the Platform model to accept string or list values for build-on and build-for architectures, resolving issue #1874. Approved by two reviewers, passed CI, and modified two files. | |
| #706 fix: improve the handling of platforms errors. | craft-application | merged | Merged fix for platform error handling. Approved by two reviewers with all CI checks passing. Resolves issue #690 and CRAFT-4369/4370. Modifies eight files with 98 additions and 239 deletions. | |
| #6069 docs: rewrite platform docs | snapcraft | merged | Squash-merged rewrite of platform documentation. Dropped core20, added core26, and renamed architectures to platforms. Fixes #6039. Approved by two reviewers with all CI checks passing. | |
| #686 fix(ProjectService): better handling for invalid platforms | craft-application | merged | Merged fix improving ProjectService invalid platform handling. Approved by two reviewers, passes CI, and updates the changelog. Incorporates targeted commits addressing review feedback from PR #683 across five files. | |
| #687 fix(commands/lifecycle): Pass the correct platform and build-for | craft-application | merged | Merged fix correcting platform and build-for parameter passing in lifecycle commands, resolving a "Multiple builds match the current platform" error. Approved by two reviewers and passed CI. |