remote-build: architecture handling
Metadata
Current evaluation
Merged architecture handling updates for remote-build. Modified one file with 36 additions and 30 deletions. Approved by one reviewer. Author noted missing CLI unit tests and plans a follow-up PR to address them.
Suggested action: —
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged architecture handling updates for remote-build. Modified one file with 36 additions and 30 deletions. Approved by one reviewer. Author noted missing CLI unit tests and plans a follow-up PR to address them. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Implements architecture handling for remote-build. Author acknowledged missing CLI unit tests and committed to a follow-up PR to add coverage once current changes stabilize. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Implemented architecture handling for remote-build. The change was merged, with the author noting that CLI unit tests will be added in a subsequent pull request. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2785 remote-build: add initial command unit tests | snapcraft | merged | Merged after adding initial unit tests for remote-build and build-on architecture handling. Review requested using an existing fixture; author updated and rebased. CI passed. | |
| #2642 remote-build: detect early build errors | snapcraft | merged | Merged. Improved remote build error detection to distinguish between pending builders and actual failures like missing config or timeouts. Approved by reviewer, passed CI, and modified 3 files (+131/-47 lines). Minor import fix applied. | |
| #4543 fix(remote): parse comma-separated architectures | snapcraft | merged | Merged after approval. Resolves #4516 by enforcing mutual exclusivity between --build-for and --build-on, and adding comma-separated architecture parsing for remote builds. Proceeded despite unrelated spread test failures. | |
| #769 feat: add the remote-build command | rockcraft | merged | Merged the remote-build command. Approved by three reviewers, the change adds 86 lines and removes 7 across nine files. Integrated by the maintainer without further comments or CI issues. | |
| #4793 fix(remote-build): don't fail with certain archs | snapcraft | merged | Merged a fix for remote-build architecture failures. The changes were retargeted to the hotfix branch per reviewer request before approval and merge. | |
| #4432 fix(remote-build): improve logging and catch exceptions | snapcraft | merged | Merged pull request improving remote build logging and exception handling. Approved by reviewers, passed CI checks, and resolves issue #4399. | |
| #1546 feat: remote build command | charmcraft | merged | Merged the remote build command feature. Approved by two reviewers and passing CI, the PR rebases prior work and adds remote build functionality. Follow-up issues were created to resolve related build-id recovery and dependency bugs. | |
| #2780 remote-build: option renames for arch, user, and accept-public-upload | snapcraft | merged | Merged. Renamed CLI options arch, user, and accept-public-upload. Adopted reviewer suggestion to rename arch to --build-on. Updated error messages and noted the change requires rebasing dependent PR 2775. | |
| #4384 refactor(remote-build): reorganize functions | snapcraft | merged | Merged a two-commit refactor reorganizing remote-build functions to prepare for issue #4322. Approved by two reviewers, passed CI checks, and modified two files with a net reduction of 40 lines. | |
| #2634 remote build: fully preserve local sources and support packaging all sources | snapcraft | merged | Merged into remote-build. Preserves local sources and enables packaging all sources. Modified 13 files (+673/-528). Iterative updates addressed review feedback, fixed tests, resolved Git import issues, and updated dependencies. Approved by one reviewer. |