fix(remotebuild): push git tags to launchpad
Metadata
Current evaluation
Merged a fix that pushes git tags to Launchpad, enabling applications to derive versions from tags. Approved by two reviewers, passed CI checks, and applied with minimal changes across two files.
Suggested action: —
No scores available.
Issue body
- [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 `tox`?
-----
Craft Application should push tags to Launchpad so that applications can derive versions from git tags.
source:
- https://matrix.to/#/!GGqzbFAUQprdPgYYCM:ubuntu.com/$EpLRd25TPvDcrTxIVTcvHQsT6XJ-_BVrYMX0-Y2p980?via=ubuntu.com&via=matrix.org&via=kde.org
- https://github.com/canonical/pebble/pull/450
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix that pushes git tags to Launchpad, enabling applications to derive versions from tags. Approved by two reviewers, passed CI checks, and applied with minimal changes across two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to push git tags to Launchpad during remote builds, allowing applications to derive versions from tags. Implemented by maintainer mr-cal to align with upstream Craft Application behavior. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #573 ci: fix bugs in Launchpad sync | fetch-service | merged | Merged a fix for the Launchpad sync workflow resolving broken tag pushes and a hardcoded username. Approved by two reviewers, passed all CI checks, and modified one file (+2/-3 lines). | |
| #224 fix(remotebuild): create the launchpad instance correctly | craft-application | merged | Approved by two reviewers and passing CI, the fix to correctly create the remotebuild launchpad instance was successfully merged. | |
| #78 fix(ci): force push to the correct launchpad repo | fetch-service | merged | Merged a one-line CI fix correcting a force push to the proper Launchpad repository. Approved by two reviewers with zero unresolved comments and no failing checks. | |
| #216 feat: enable launchpad hosted repos | fetch-service | merged | Merged changes enabling inspection of Launchpad-hosted git repositories. Approved by two reviewers with zero unresolved comments. Updates span five files (+23/-5 lines). | |
| #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. | |
| #219 fix: add launchpad build types and states | craft-application | merged | Merged a maintainer fix adding launchpad build types and states. The two-line change in one file passed CI and received approval from two reviewers before integration. | |
| #230 fix: make launchpad client always create relevant files | craft-application | merged | Merged a fix ensuring the launchpad client always creates relevant files. Approved by two reviewers with passing CI checks. The change adds 138 lines across three files. | |
| #244 fix: handle multiple types of Launchpad error when getting projects | craft-application | merged | Merged a fix that handles multiple types of Launchpad errors when retrieving projects. Approved by two reviewers, passed CI checks, and applied minimal changes (+2/-2 lines in one file.) | |
| #236 fix(launchpad): correct the uploading build state | craft-application | merged | Merged a fix correcting an incorrect string in the buildstate property for Launchpad builds. The change was approved by reviewers, passed CI checks, and required only a single line modification. |