← Back to issue list

Added a linter to verify that the metadata is ok (CRAFT-322).

View original Github issue

Metadata

Project
charmcraft
Number
#449
Type
pull request
State
merged
Author
facundobatista
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Note that linters are still NOT enforced when packing (will do that in the next branch when I add the --force option to pack anyway). Also note that this PR depends on the "analint 06" one.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged a metadata validation linter. The change depends on a separate PR and defers enforcement during packing. The implementation was later incorporated into a broader effort tracked in issue #27.
qwen3.6-35b-a3b-mtp-q6 Merged a PR adding a metadata linter. Enforcement during packing is deferred, and the change depends on the analint 06 PR. It partially addresses the first item in issue #27.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5155 Metadata Linter: A new linter in snapcraft to lint about missing metadata fields snapcraft closed Implemented a metadata linter to warn about missing fields like license, title, and contact info. The enhancement was completed and merged via pull request #5610.
74%
#451 Handle linters when building (CRAFT-325). charmcraft merged Merged implementation for handling linters during build. The change surfaces errors and warnings, failing unless --force is passed to pack. Approved by two reviewers and depends on PR #449.
72%
#5610 feat(metadata-linter): impl new linter snapcraft merged 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.
70%
#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.
70%
#445 Use linters' analyze from build command (CRAFT-318) charmcraft merged Merged. Integrated linter analysis into the build command to log attributes and update manifest creation. Fixed a missing integration test in checkers/analyze. Approved by reviewers and passed CI checks.
70%