fix: properly ignore source-code field in metadata linter
Metadata
Current evaluation
Merged to resolve issue #5794 by properly ignoring the source-code field in the metadata linter. Approved by two reviewers, passed all CI checks, and verified working.
Suggested action: —
No scores available.
Issue body
- [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`?
---
Fixes #5794. Does a little fly-by typing work while I'm in this file, as well.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to resolve issue #5794 by properly ignoring the source-code field in the metadata linter. Approved by two reviewers, passed all CI checks, and verified working. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix updating the metadata linter to properly ignore the source-code field, resolving issue #5794. Verified by a user after building from the branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix that properly ignores the source-code field in the metadata linter, resolving issue #5794. Verified by a user who confirmed the build resolves the problem. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #96 many: fix source ignore patterns (CRAFT-375) | craft-parts | merged | Merged fix for source ignore patterns preventing locally generated files from being pulled as sources. Updates allow custom ignore patterns for outputs and correctly ignore work directories. Approved by two reviewers with passing CI checks. | |
| #229 fix: remove inspecting text/html in sourcecraft inspector | fetch-service | merged | Merged to resolve issue #228 by removing text/html inspection in the sourcecraft inspector, preventing accidental processing of non-git requests. Approved by two reviewers and passed CI checks. | |
| #2222 style: fix linting issues | charmcraft | merged | Merged following approval by two reviewers. Resolved linting issues across three files with minor style adjustments (+8/-9 lines). No review comments or CI checks were required. | |
| #20 style: fix linting issues | craft-artifacts | merged | Merged after passing all CI checks and receiving approval from two reviewers. The change resolves linting issues across four files with a +11/-5 line diff, updating code style compliance. | |
| #969 fix: decode filenames | craft-application | merged | Merged to resolve issue #968 by implementing filename decoding. Approved by two reviewers with passing CI checks. The change updates two files, adding 35 lines and removing 1. | |
| #328 fix(inspectors/deb): remove incorrect metadata mapping | fetch-service | merged | Merged fix removing incorrect deb inspector metadata mapping. Eliminates assumptions linking Upstream-Contact and maintainer fields to author info to prevent false positives. Approved by two reviewers, passed CI, and resolves #324. |