← Back to issue list

chore!: remove legacy remote builder

View original Github issue

Metadata

Project
snapcraft
Number
#5994
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Run Manual Spread
Created
Updated
Closed

Current evaluation

Merged changes removing the legacy remote builder, resolving issue #5676. Approved by three reviewers and passing CI checks. The update deletes 3,200 lines across 30 files to clean up deprecated build infrastructure.

Suggested action: —

No scores available.

Issue body

Removes the legacy remote builder. Fixes #5676 (SNAPCRAFT-1197) --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [x] I've added or updated any relevant documentation. - [x] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged changes removing the legacy remote builder, resolving issue #5676. Approved by three reviewers and passing CI checks. The update deletes 3,200 lines across 30 files to clean up deprecated build infrastructure.
qwen3.6-35b-a3b-mtp-q6 — — Merged by maintainer mr-cal, this change removes the legacy remote builder, resolving issue #5676 and ticket SNAPCRAFT-1197. The update passed all lint and test checks before integration.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of the legacy remote builder, resolving issue #5676. Maintainer mr-cal integrated the change, eliminating deprecated build infrastructure and updating documentation.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#6010 chore!: remove remote-build service override snapcraft merged Merged cleanup removing the remote-build service override and manual spread tests. This follow-up addresses overlooked work from PR #5994. Approved by two reviewers, modifying seven files.
82%
#5523 chore: add deprecation warning for legacy remote builder snapcraft merged Merged changes adding a deprecation warning for core22 snaps using the legacy remote builder. Approved by three reviewers with passing CI checks. Resolves issue #5427.
78%
#667 chore!(models): remove BuildInfo and BuildPlanner craft-application merged Merged a breaking change removing the BuildInfo and BuildPlanner models across nine files, deleting 430 lines and adding 36. Approved by two reviewers and passed CI checks.
77%
#2682 chore: delete some dead code charmcraft open Removes 19 lines of dead code in one file, a follow-up to #2681 (remote-build project name fix). Approved by a maintainer, CI mostly green with a few failing/pending snap tests.
75%
#6035 chore!: remove deprecated commands snapcraft merged Merged after maintainer approval and a clean rebase. The changes remove deprecated commands and options, resolving issue #5674.
75%
#4383 chore: rename remote-build test snapcraft merged Merged a code cleanup that renamed the test module to match the source file remote.py. Approved by two reviewers, passed CI checks, and successfully integrated into the main branch.
75%
#5821 chore(legacy): remove script entry point snapcraft merged Merged removal of a legacy script entry point causing user confusion. Approved by two reviewers, passed CI checks, and deleted a single line in one file to clean up the codebase.
74%
#4345 chore: remove unused list_registered legacy code snapcraft merged Merged changes removing unused legacy list_registered code, eliminating final references to Price. Approved by two reviewers, passed CI, and fixes #4296. The commit modifies four files, deleting 49 lines.
73%
#580 chore: update bld and store api urls fetch-service merged Merged after three approvals, updating the default configuration to support non-staging API URLs and resolving issue #571. CI completed with one scanner warning. Changes added five lines to one file.
72%
#1776 chore: remove unused code charmcraft merged Merged after removing 17,571 lines of unused code across 36 files. Temporarily set to draft for additional cleanup, then approved by two reviewers and merged.
72%