← Back to issue list

models: add metadata model

View original Github issue

Metadata

Project
craft-application
Number
#21
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged into main after reviewer approval and passing CI. The changes introduce a metadata model per CRAFT-1751, modifying two files. Commits were squashed, diff coverage reached 100%, and overall coverage remained unchanged.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? ----- CRAFT-1751 Please ignore the commit history, this is getting squashed anyway :grimacing:

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged into main after reviewer approval and passing CI. The changes introduce a metadata model per CRAFT-1751, modifying two files. Commits were squashed, diff coverage reached 100%, and overall coverage remained unchanged.
qwen3.6-35b-a3b-mtp-q6 Merged into main, introducing a metadata model for CRAFT-1751. The commit was squash-merged by maintainer lengau while maintaining 100% test coverage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#19 models: add base metadata model craft-application closed Merged to add a base metadata model, resolving CRAFT-1751. The change added 1092 lines across 28 files with 97.36% diff coverage, though overall coverage decreased by 2.64%.
88%
#3700 meta: adopt metadata info snapcraft merged Merged into snapcraft/7.0 following reviewer approval and passing CI. Updates project fields with external metadata, addressing CRAFT-1036. The change modifies 21 files, increases coverage by 0.01%, and resolves the metadata adoption task.
75%
#3555 project: validate metadata links (CRAFT-369) snapcraft merged Merged into master after receiving two reviewer approvals and passing CI checks. The change implements project metadata link validation, adding 215 lines across four files.
75%
#3552 meta: support for metadata links (CRAFT-370) snapcraft merged Merged into master following approval and passing CI checks. The update implements metadata link support, maintaining 100% diff coverage. Review comments were addressed prior to final integration.
73%
#473 feat: models for the craft manifest craft-application merged Merged three new models for craft manifest metadata: ProjectManifest, SessionArtifactManifest, and CraftManifest. Approved by two reviewers, passed CI checks, and added 439 lines across seven files.
73%
#60 models: export classes for easier consumption craft-store merged Merged changes exporting model classes to simplify consumption. Approved by one reviewer, passed CI, and added nine lines to one file. Addresses CRAFT-1239.
71%
#20 models: add project model craft-application merged Merged pull request adding a new project model. Approved and passed CI. Despite a minor coverage drop and a request to rebase and fix linting, the changes were integrated into main.
71%
#23 app: add AppMetadata data class for holding info about the app craft-application merged Merged into main after approval and CI checks. Adds an AppMetadata data class for application info, resolving CRAFT-1755. Changes added 68 lines across 2 files with 100% test coverage and no impact on overall coverage.
70%