← Back to issue list

feat(metadata-linter): impl new linter

View original Github issue

Metadata

Project
snapcraft
Number
#5610
Type
pull request
State
merged
Author
soumyaDghosh
Labels
—
Created
Updated
Closed

Current evaluation

Merged implementation of a new metadata linter enforcing rank-based checks, adding an INFO result type, and supporting the icon field. Includes unit and spread tests. Closes #5544 and #5155 after three reviewer approvals.

Suggested action: —

No scores available.

Issue body

1. added the new linter 2. lints based on ranks 3. added unit tests and spread tests for the new linter 4. added icon field to SnapMetadata from Project model 5. added a new LinterResult value named INFO - [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint`? - [x] Have you successfully run `make test`? --- closes #5544 #5155

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged implementation of a new metadata linter enforcing rank-based checks, adding an INFO result type, and supporting the icon field. Includes unit and spread tests. Closes #5544 and #5155 after three reviewer approvals.
qwen3.6-35b-a3b-mtp-q6 — — Merged an external contribution implementing a new metadata linter with rank-based validation, unit and spread tests, an icon field for SnapMetadata, and a new INFO result type. Closes #5544 and #5155 after internal review.
qwen3.6-35b-a3b-mtp-q6 — — Merged implementation of a new metadata linter with rank-based checks, unit and spread tests, an icon field in SnapMetadata, and a new INFO result type. Closes #5544 and #5155 after internal review and final approval.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#449 Added a linter to verify that the metadata is ok (CRAFT-322). charmcraft merged Merged after approval, adding a linter to verify metadata. The change depends on a separate analint PR and introduces validation checks, though enforcement during packing is deferred to a future release.
70%