fix: use craft-application clean command
Metadata
Current evaluation
Closed pull request implementing a fix to use the craft-application clean command. The change modifies five files, removing 300 lines and adding 10. No review comments or CI checks were recorded. The PR was closed without explicit merge status.
Suggested action: —
No scores available.
Issue body
Requires #1419
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Closed pull request implementing a fix to use the craft-application clean command. The change modifies five files, removing 300 lines and adding 10. No review comments or CI checks were recorded. The PR was closed without explicit merge status. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed and abandoned pending PR #1419. No merge activity or comments were recorded. The update to use the craft-application clean command was not merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed and abandoned. The pull request depended on #1419, which was never merged, leaving the fix to use the craft-application clean command unapplied. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1571 refactor: remove unused clean command implementation | charmcraft | merged | Merged refactoring that removes the unused custom clean command and associated regression tests, as the project now uses the craft-application clean command. Approved by three reviewers, passed CI, and modified four files. | |
| #1563 fix: breaking changes from craft-application main | charmcraft | merged | Merged to resolve breaking changes from craft-application main. Approved by two reviewers, the update modified five files (+46/-26 lines) and was successfully integrated into the codebase. | |
| #4659 fix: update to craft-application latest | snapcraft | merged | Merged. Consolidates fixes from PRs #4616, #4643, and #4651 to update craft-application to the latest version and accommodate its main branch breaking changes. Approved by two reviewers, passed CI, and successfully merged. | |
| #1386 fix: final spread test fixes for craft-application | charmcraft | merged | Merged after approval by two reviewers. Resolves final spread test issues for craft-application across seven files, removing 127 lines and adding 104. Changes integrated into the main branch. | |
| #1520 refactor: remove old "init" command implementation | charmcraft | merged | Merged a refactoring PR removing the old "init" command after the craft-application transition. Approved by two reviewers with passing CI, the change modified four files, adding one line and deleting 248. | |
| #1454 build: update craft-application | charmcraft | merged | Merged after updating the craft-application dependency and adjusting test constants. Approved by two reviewers with no unresolved comments. Changes affect three files with minor line adjustments. | |
| #1704 craft-application migration cleanup: pack command | charmcraft | merged | Merged cleanup PR removing an outdated pack command retained for comparison during the craft-application migration. Approved by two reviewers with passing CI, the change deletes 1309 lines across six files. | |
| #856 refactor: use craft-application 5 | rockcraft | merged | Merged after three reviewers approved. The change refactored the codebase to adopt craft-application version 5, updating 49 files with 959 additions and 915 deletions. All checks passed. | |
| #1399 feat: craft-application rebase | charmcraft | merged | Merged the craft-application rebase, altering 109 files with 7,783 additions and 755 deletions. Approved by two reviewers, the update was deemed functional for integration, with remaining issues deferred to future pull requests. | |
| #1340 feat(craft-app): move init command | charmcraft | merged | Merged maintainer PR moving the craft-app init command. Includes required lint fixes, approved by three reviewers, and modifies 18 files (+500/-343 lines). |