← Back to issue list

chore: delete unused file

View original Github issue

Metadata

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

Current evaluation

Merged PR #6079 as commit 75686f3da, deleting a three-line file unused since Snapcraft moved to a pyproject-based setup.

Resolution: closed resolved

Reason: PR #6079 was merged with all CI checks passing, removing the obsolete file in commit 75686f3da.

Evaluated by: mr-cal-bot

No scores available.

Issue body

I noticed while reviewing #6075 that we didn't use this file at all since moving to a pyproject-based setup (and perhaps even before that?) --- - [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/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #6079 as commit 75686f3da, deleting a three-line file unused since Snapcraft moved to a pyproject-based setup.
stealth/space-bunny-alpha — closed resolved Merged PR #6079 as commit 75686f3da, deleting a three-line file unused since Snapcraft moved to a pyproject-based setup.
stealth/space-bunny-alpha — closed resolved Merged PR #6079 as commit 75686f3da, deleting a three-line file unused since the move to a pyproject-based setup; all CI checks passed.
stealth/space-bunny-alpha — closed resolved Merged PR #6079 as 75686f3da, deleting a three-line file unused since Snapcraft moved to a pyproject-based setup.
stealth/space-bunny-alpha — closed resolved Merged #6079 as commit 75686f3da, deleting an unused three-line file left over from Snapcraft's pyproject-based setup; all CI checks passed.
stealth/space-bunny-alpha — closed resolved PR #6079 was merged as commit 75686f3da, deleting a three-line file unused since Snapcraft's pyproject-based setup.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged a maintainer's change to delete an unused file that became obsolete following the migration to a pyproject-based setup. No discussion or additional testing was required.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#6075 (confidence 98%)

    The unused file was identified while the author was reviewing #6075.

Related issues

Issue Project State Summary Similarity
#1776 chore: remove unused code charmcraft merged Merged PR #1776 as commit 027696be, deleting 17,571 lines of unused code across 36 files after maintainer review and approval.
79%
#6470 chore: remove unused errors snapcraft closed PR removing unused SnapcraftError subclasses was closed unmerged; superseded by reworking the branch layout and pushing directly to canonical/snapcraft.
77%
#6241 chore: remove snapcraft_legacy snapcraft merged Merged #6241 via commits ffb164b7a and 995f0a3bc, deleting snapcraft_legacy, obsolete specs, and legacy-only dependencies after its entry points were removed.
76%
#6074 chore!: remove default command snapcraft merged PR #6074 merged as db7c53f21, removing bare snapcraft packing for #5673; #6178 later reverted the change in e5aaf0683.
76%
#2444 chore: clean up unused/unnecessary files charmcraft merged Merged maintenance PR #2444 (commit 6b94f660) removing 286 lines across 10 unused/unnecessary files after two approvals; macOS smoke-test jobs failed but did not block the merge.
76%
#6474 chore: remove unused errors snapcraft merged Merged PR removing three unused SnapcraftError subclasses (InvalidArchitecture, ArchAllInvalid, ProjectMissing) and a stale comment in services/init.py; no behavior change.
76%
#6469 chore: remove unused _read_config snapcraft closed Closed without merging; superseded by a reworked branch pushed directly to canonical/snapcraft under work/refactor/*.
76%
#1078 chore!: remove deprecated APIs craft-application merged Merged PR #1078 as 3334f13e, removing eight deprecated APIs and fixing #1063. All CI checks passed, with local Snapcraft verification.
75%
#4803 chore(merge): '8.2.7' into 'main' snapcraft merged Merged Snapcraft 8.2.7 into main via approved PR #4803 at commit 1d873e817; 21 files changed (+475/-20).
75%
#6035 chore!: remove deprecated commands snapcraft merged Merged as 3623601d7, removing deprecated Snapcraft commands and options and resolving #5674 (SNAPCRAFT-1196).
75%