← Back to issue list

refactor: migrate metrics command

View original Github issue

Metadata

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

Current evaluation

Merged the refactor migrating the metrics command from the legacy codebase to the new implementation. The change modified 21 files, adding 683 lines and removing 1002, resolving SNAPCRAFT-1319 after successful review and CI validation.

Suggested action:

No scores available.

Issue body

Migrates the `metrics` command from the legacy codebase into the new. SNAPCRAFT-1319 --- - [ ] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [ ] 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. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged the refactor migrating the metrics command from the legacy codebase to the new implementation. The change modified 21 files, adding 683 lines and removing 1002, resolving SNAPCRAFT-1319 after successful review and CI validation.
qwen3.6-35b-a3b-mtp-q6 Merged the migration of the metrics command from the legacy codebase to the new implementation. Resolves SNAPCRAFT-1319. The refactoring was approved and integrated by maintainer bepri.
qwen3.6-35b-a3b-mtp-q6 Merged the migration of the metrics command from the legacy codebase to the new architecture, resolving SNAPCRAFT-1319. The refactor was completed by maintainer bepri and integrated directly into the main branch.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#6144 refactor: migrate keys snapcraft merged Squash merged the migration of the keys command from snapcraft_legacy. Approved by two reviewers, the change modified 12 files with a net reduction of 157 lines. Resolves SNAPCRAFT-1315.
75%
#6145 refactor: migrate gated command snapcraft merged Merged the refactoring that migrates the gated command from snapcraft_legacy. Approved by two reviewers and passed most CI checks. Resolves SNAPCRAFT-1314.
71%
#5372 docs: migrate 'metrics and statistics' snapcraft merged Merged documentation migration for metrics and statistics. Approved by two reviewers, the update modifies six files with 193 additions and 33 deletions.
71%
#6218 refactor: migrate promote command snapcraft merged Migrated the promote command from snapcraft_legacy to the main codebase as a near-verbatim port to preserve complex legacy behaviors. Approved by two reviewers and merged after passing most CI checks, despite one store integration test failure.
70%