← 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 as commit 98bb42c3b, migrating the metrics command from Snapcraft’s legacy codebase to the new command architecture for SNAPCRAFT-1319.

Resolution: closed resolved

Reason: PR #6207 was approved and merged as commit 98bb42c3b. Follow-up PR #6217 subsequently fixed a metrics display typo affecting snaps with no data.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged as commit 98bb42c3b, migrating the metrics command from Snapcraft’s legacy codebase to the new command architecture for SNAPCRAFT-1319.
stealth/space-bunny-alpha — closed resolved Merged PR #6207 as commit 98bb42c3b, migrating the metrics command from the legacy Snapcraft codebase to the new implementation for SNAPCRAFT-1319.
stealth/space-bunny-alpha — closed resolved Merged PR #6207 as commit 98bb42c3b, completing the migration of the metrics command from Snapcraft’s legacy codebase to the new implementation for SNAPCRAFT-1319.
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.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#6217 (confidence 99%)

    Merged follow-up fixing metrics display for snaps without data.

Related issues

Issue Project State Summary Similarity
#5372 docs: migrate 'metrics and statistics' snapcraft merged Merged in canonical/snapcraft#5372 at commit f0c4096a8, migrating the metrics and statistics documentation and updating six files (+193/-33).
75%
#6145 refactor: migrate gated command snapcraft merged Merged PR #6145 as commit 21091b932, migrating the gated command from snapcraft_legacy and resolving SNAPCRAFT-1314.
73%
#3560 cli: introduce metrics command (CRAFT-387) snapcraft merged Merged PR #3560, introducing snapcraft metrics with JSON/table output, required name and format arguments, and optional date ranges.
72%
#6144 refactor: migrate keys snapcraft merged PR #6144 merged the migration of the keys command out of snapcraft_legacy, completing SNAPCRAFT-1315 in commit 30c9fcd19.
72%
#6218 refactor: migrate promote command snapcraft merged Merged the promote command migration from snapcraft_legacy into the main codebase in commit 4012735f6, preserving legacy behavior.
72%