← Back to issue list

chore(legacy): remove script entry point

View original Github issue

Metadata

Project
snapcraft
Number
#5821
Type
pull request
State
merged
Author
bepri
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `make test`? --- This removes an old legacy entry point that caused some confusion (see #5778).

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged maintainer bepri's cleanup to remove a legacy script entry point that caused confusion, addressing issue #5778. The change was approved and merged without additional comments.
qwen3.6-35b-a3b-mtp-q6 Merged maintainer cleanup removing a legacy script entry point that caused user confusion, resolving issue #5778. The change eliminates outdated code paths without affecting core functionality.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#392 chore: remove codeowners starbase merged Merged after approval from two reviewers and passing all CI checks. The change removes the CODEOWNERS file, deleting 9 lines from a single file as discussed in standup.
77%
#1765 chore: remove classic entrypoint charmcraft merged Merged after approval by two reviewers. Removed a leftover classic entrypoint from the craft-application changeover, modifying seven files.
76%
#907 chore(merge): 2.1.3 into main craft-parts merged Merged version 2.1.3 into main following approval from two reviewers and passing CI checks. The change updated seven files, adding 36 lines and removing 10.
76%
#6079 chore: delete unused file snapcraft merged Merged a pull request removing an unused file following a migration to a pyproject-based setup. The change was approved by two reviewers, passed all CI checks, and deleted three lines from a single file.
76%
#6241 chore: remove snapcraft_legacy snapcraft merged Merged removal of the snapcraft_legacy module, old specs folder, and dependencies after verifying no remaining entry points. Approved by two reviewers; noted unrelated integration test failures.
76%
#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.
75%
#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.
75%
#923 chore(merge): 2.1.4 into main craft-parts merged Merged release 2.1.4 back into the main branch. Approved by two reviewers with passing CI checks. The update modified 10 files, adding 49 lines and removing 16.
75%
#5287 chore: delete empty shellcheckrc file snapcraft merged Merged PR removing an accidentally committed empty shellcheckrc file. Approved by two reviewers and passed CI, successfully cleaning the repository root.
75%
#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.
74%