← Back to issue list

project_loader: support architectures for CI

View original Github issue

Metadata

Project
snapcraft
Number
#2080
Type
pull request
State
merged
Author
kyrofa
Labels
—
Created
Updated
Closed

Current evaluation

PR #2080 by kyrofa was merged (commit c60ac3891), adding CI-friendly `architectures` syntax with build-on/run-on lists to project_loader, resolving issue #1685.

Resolution: closed resolved

Reason: The PR was approved and merged as commit c60ac3891 ('project_loader: support architectures for CI (#2080)'), implementing the CI-friendly architectures proposal and closing snapcraft#1685; the syntax persists today (later extended with build-for in c9822a38c / #5606).

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? ----- This PR resolves #1685 by changing the `architectures` keyword according to the new CI-friendly proposal [here](https://forum.snapcraft.io/t/4675). This adds support for new syntax to that keyword: architectures: - build-on: [<build arch 1>, <build arch 2>] run-on: [<run arch 1>, <run arch 2>] Please see the proposal for more details.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #2080 by kyrofa was merged (commit c60ac3891), adding CI-friendly `architectures` syntax with build-on/run-on lists to project_loader, resolving issue #1685.
stealth/space-bunny-alpha — closed resolved PR #2080 by kyrofa merged, resolving #1685: the `architectures` keyword gained the CI-friendly `build-on`/`run-on` list syntax (commit c60ac3891); `run-on` was later renamed to `build-for`.
stealth/space-bunny-alpha — closed resolved CI-friendly architectures syntax was merged in #2080, adding build-on and run-on lists and resolving #1685 (commit c60ac3891).
qwen/qwen3.6-35b-a3b — — Implemented CI-friendly architectures keyword syntax with build-on and run-on fields. Merged after reviewer approval and passing CI checks. Resolves #1685. Minor discussions on validation error messages and test timeouts did not block the merge.
qwen3.6-35b-a3b-mtp-q6 — — Merged to implement CI-friendly architecture syntax using build-on and run-on fields in the project loader, resolving issue #1685. Validation error improvements for typos were noted but deferred.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: snapcraft#1685 (confidence 97%)

    Issue explicitly resolved by this PR; closed as done, with build.snapcraft.io adoption left as follow-up

  • Related To: snapcraft#5606 (confidence 70%)

    Later commit c9822a38c added build-for alongside the build-on/run-on architectures syntax introduced here

Related issues

No related issues found above the similarity threshold.