remote-build: package up local sources with source-type 'git'
Metadata
Current evaluation
Merged after approval. Implements remote-build packaging for local sources with source-type 'git'. Approved by one reviewer with zero unresolved comments. Changes modify 2 files (+10/-3 lines).
Suggested action: —
No scores available.
Issue body
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval. Implements remote-build packaging for local sources with source-type 'git'. Approved by one reviewer with zero unresolved comments. Changes modify 2 files (+10/-3 lines). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged an update enabling remote-build to package local sources when using the 'git' source type. The maintainer-authored change was integrated directly into the main branch without additional review comments or revisions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged remote-build update that packages local sources for the 'git' source type. Integrated into the main branch by maintainer cjp256. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #310 fix(remote-build): enable git repo check | craft-application | merged | Successfully merged the fix enabling the git repository check for remote builds. Approved by two reviewers, the update modifies two files with 18 additions and 1 deletion, resolving the build configuration issue. | |
| #2947 remote-build: pass through 'source-subdir' property | snapcraft | merged | Merged. Fixes remote-build incorrectly stripping the 'source-subdir' property by passing it through. Includes test coverage for source property handling. Approved and passed CI. | |
| #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. | |
| #2837 remote-build: gpg-signing and usability fixes | snapcraft | merged | Merged changes disabling GPG signing for git commits to fix remote-build failures. Improves error messaging and commit formatting. Approved by reviewer, passed CI, and integrated into the codebase. | |
| #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. | |
| #2500 cli: add remote build | snapcraft | merged | Merged. Adds remote build capability to Launchpad, dispatching source trees for remote compilation and retrieving resulting snap packages or logs. Approved by reviewers, passed CI, and integrated despite a minor test coverage decrease. | |
| #2564 remote_build: handle git push in detached head state | snapcraft | merged | Merged fix for remote_build that resolves a detached HEAD exception by returning HEAD instead of failing branch name retrieval. Approved by reviewer and passed CI checks. | |
| #4460 fix: push tags for remote-build | snapcraft | merged | Merged after approval from two reviewers. The fix enables pushing tags for remote-build, maintaining 89.18% overall test coverage with 100% on modified lines. Changes span four files, adding 57 lines and removing 5. |